-
Notifications
You must be signed in to change notification settings - Fork 24
Show 'claims_supported' in discovery document #137
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
Comments
Keys from 'claim to attribute translate table' from module config file could be used, however, claim names can have type designations at the beginning... |
I think (even without dealing with the type issue) you would need to get the claims from Then maybe something like this to remove the types $claims = [];
foreach ($this->translationTable as $claimWithType) {
list($type, $claim) = $this->getTypeAndClaimName($claimWithType);
$claims[] = $claim;
} Overall I agree with your point in #138 that maybe a different syntax for indicating type would make the code and data structures easier. |
Great, thanks for sharing this option! |
Note to self that OpenID Federation certification tests show warning for this, as this is the recommended feature to implement by the OpenID Discovery spec:
|
Find a way to show which claims are supported by OP using 'claims_supported' property in discovery document as per https://openid.net/specs/openid-connect-discovery-1_0.html
The text was updated successfully, but these errors were encountered: