Skip to content

Updates to ADFS MFA keycredentials management #8049

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fredentoft
Copy link

@fredentoft fredentoft commented May 8, 2025

Fixed objecttype of $newKey for credentials.
Added hostname to displayname of the keyCredentials to improve lifecycle management of them.
Added note about a bug in powershell module.
Added information regarding removal of unused and expired credentials.

Added hostname to displayname of the keyCredentials to improve lifecycle management of them.
Added note about a bug in powershell module.
Added information regarding removal of unused and expired credentials.
@fredentoft
Copy link
Author

@microsoft-github-policy-service agree

Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit 571bb74:

✅ Validation status: passed

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

For more details, please refer to the build report.

For any questions, please:

@v-regandowner
Copy link
Contributor

#assign: @robinharwood, @Xelu86

@robinharwood @Xelu86
Can you review the proposed changes?

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Copy link
Contributor

Users robinharwood are already assigned.

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label May 8, 2025
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.

4 participants