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
Note about Token Store for Claims Mapping to work #107220
Conversation
If the token store is disabled (non-default) then the AAD ID token (`X-MS-TOKEN-AAD-ID-TOKEN`) is not sent and at least the big `Microsoft.Identity.Web` SDK can't map the weird `X-MS-CLIENT-PRINCIPAL` (non-JWT) to a ClaimsPrincipal
@nis-spiir : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit 8471669:
|
File | Status | Preview URL | Details |
---|---|---|---|
articles/app-service/configure-authentication-user-identities.md | Details |
articles/app-service/configure-authentication-user-identities.md
- Line 31, Column 51: [Warning: hard-coded-locale - See documentation]
Link 'https://learn.microsoft.com/en-us/azure/app-service/overview-authentication-authorization#token-store' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
- Line 31, Column 51: [Suggestion: docs-link-absolute - See documentation]
Absolute link 'https://learn.microsoft.com/en-us/azure/app-service/overview-authentication-authorization#token-store' will be broken in isolated environments. Replace with a relative link.
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
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.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
Learn Build status updates of commit 850db4e: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Can you review the proposed changes? When the changes are ready for publication, add a #label:"aq-pr-triaged" |
#sign-off |
Due to the upcoming conference, all merging and publishing are on hold until the morning of 5/23. This PR is ready for merge on 5/23. #label: "event-triaged-to-10AM" |
If the token store is disabled (non-default) then the AAD ID token (
X-MS-TOKEN-AAD-ID-TOKEN
) is not sent and at least the bigMicrosoft.Identity.Web
SDK can't map the weirdX-MS-CLIENT-PRINCIPAL
(non-JWT) to a ClaimsPrincipal