Skip to content
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

Add x5c Header when Acquiring PoP Tokens #568

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

JorgeDaboub
Copy link
Contributor

@JorgeDaboub JorgeDaboub commented Dec 7, 2024

If an SPN uses an SNI certificate for authentication, the x5c header must be included in the request. Omitting this header will cause the authentication call to fail. However, if the credentials are not a certificate, no error occurs, and the process continues without issues.

Failing call:
Pipeline Run

Error:
AADSTS700027: The certificate with the identifier used to sign the client assertion is not registered on the application. [Reason: The key was not found.] Thumbprint of the key used by the client:

Successful Run with Fix Applied:
Pipeline Run

Copy link
Contributor

@weinong weinong left a comment

Choose a reason for hiding this comment

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

LGTM

@weinong weinong merged commit c09e0cb into Azure:main Dec 9, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants