Skip to content

Conversation

@ChristianWeisse
Copy link

updating links for Azure MFA proofup page
replacing legacy endpoint https://account.activedirectory.windowsazure.com/Proofup.aspx with recommended endpoint
aka.ms/mfasetup

update to role requirements for Azure MFA Client configuration. Application Admin is not enough (ICM 666354843)

updating links for Azure MFA proofup page 
replacing legacy endpoint https://account.activedirectory.windowsazure.com/Proofup.aspx
with recommended endpoint 
aka.ms/mfasetup

update to role requirements for Azure MFA Client configuration.
Application Admin is not enough  (ICM 666354843)
@ChristianWeisse ChristianWeisse requested a review from a team as a code owner November 5, 2025 16:31
@ChristianWeisse ChristianWeisse requested review from robinharwood and removed request for a team November 5, 2025 16:31
@prmerger-automator prmerger-automator bot requested a review from Justinha November 5, 2025 16:31
@prmerger-automator
Copy link
Contributor

@ChristianWeisse : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit f6706a1:

💡 Validation status: suggestions

File Status Preview URL Details
WindowsServerDocs/identity/ad-fs/operations/configure-ad-fs-and-azure-mfa.md 💡Suggestion Details

WindowsServerDocs/identity/ad-fs/operations/configure-ad-fs-and-azure-mfa.md

  • Line 78, Column 3: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: /powershell/microsoftgraph/installation?view=graph-powershell-1.0

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@v-dirichards v-dirichards requested a review from Copilot November 5, 2025 21:20
@v-dirichards
Copy link
Contributor

@robinharwood, @Xelu86
Can you review the proposed changes?

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team
#assign: @robinharwood, @Xelu86

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Nov 5, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the AD FS and Azure MFA configuration documentation to reflect current URLs, prerequisites, and best practices. The changes modernize outdated references and improve clarity.

  • Updated legacy Azure MFA setup URLs to the current https://aka.ms/mfasetup shortlink
  • Modernized prerequisites to include Windows Server 2016 or later and Microsoft Graph PowerShell
  • Improved formatting and clarity in the Azure Government cloud configuration section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- Enterprise administrator credentials to configure the AD FS farm for Microsoft Entra multifactor authentication.
- You'll need either an account that has the [Application Administrator](/entra/identity/role-based-access-control/permissions-reference#cloud-application-administrator) role on your instance of Microsoft Entra ID to configure it by using PowerShell.
- [Microsoft Graph PowerShell Module](/powershell/microsoftgraph/installation?view=graph-powershell-1.0).
- Active Directory Users must be synchronized with the Entra ID Tenant (multiple tenants are not supported)
Copy link

Copilot AI Nov 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing 'Microsoft' before 'Entra ID'. The product name should be 'Microsoft Entra ID' for consistency with other references in the document.

Suggested change
- Active Directory Users must be synchronized with the Entra ID Tenant (multiple tenants are not supported)
- Active Directory Users must be synchronized with the Microsoft Entra ID Tenant (multiple tenants are not supported)

Copilot uses AI. Check for mistakes.
//Provide a message and redirect to Azure AD MFA Registration Url
var mfaRegisterUrl = "https://account.activedirectory.windowsazure.com/proofup.aspx?proofup=1&whr=" + domain_hint;
var mfaRegisterUrl = "https://aka.ms/mfasetup?domain_hint=" + domain_hint;
Copy link

Copilot AI Nov 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The URL parameter format is incorrect. The original URL used 'whr=' parameter, but this change uses 'domain_hint=' without a '&' separator after the base URL. It should be 'https://aka.ms/mfasetup?whr=' to maintain the same parameter pattern, or if domain_hint is the correct parameter for the new URL, there should be documentation confirming this change in behavior.

Suggested change
var mfaRegisterUrl = "https://aka.ms/mfasetup?domain_hint=" + domain_hint;
var mfaRegisterUrl = "https://aka.ms/mfasetup?whr=" + domain_hint;

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants