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

updated to reflect that FQDN should include prefix for OIDC compliance. #3486

Conversation

mooreds
Copy link
Contributor

@mooreds mooreds commented Jan 29, 2025

https://openid.net/specs/openid-connect-discovery-1_0.html#ProviderMetadata says

the issuer is 'REQUIRED. URL using the https scheme with no query or fragment components that the OP asserts as its Issuer Identifier. If Issuer discovery is supported (see Section 2), this value MUST be identical to the issuer value returned by WebFinger. This also MUST be identical to the iss Claim value in ID Tokens issued from this Issuer. '

We should recommend that the user use the full URL, not just the FQDN (which does not include https://)

I also updated the image, but decided to submit that as #3488 .

https://openid.net/specs/openid-connect-discovery-1_0.html#ProviderMetadata says

the issuer is 'REQUIRED. URL using the https scheme with no query or fragment components that the OP asserts as its Issuer Identifier. If Issuer discovery is supported (see Section 2), this value MUST be identical to the issuer value returned by WebFinger. This also MUST be identical to the iss Claim value in ID Tokens issued from this Issuer. '
@mooreds mooreds requested review from a team as code owners January 29, 2025 18:03
@mooreds mooreds mentioned this pull request Jan 29, 2025
2 tasks
@mooreds mooreds merged commit ec29960 into main Jan 29, 2025
4 checks passed
@mooreds mooreds deleted the mooreds/update-tenant-oauth-tab-to-mention-oidc-compatible-url-for-issuer branch January 29, 2025 19:42
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