Skip to content
This repository has been archived by the owner on Feb 11, 2025. It is now read-only.

Role claim type should be "roles" #517

Closed
ahaeber opened this issue Jun 12, 2023 · 0 comments · Fixed by #519
Closed

Role claim type should be "roles" #517

ahaeber opened this issue Jun 12, 2023 · 0 comments · Fixed by #519

Comments

@ahaeber
Copy link

ahaeber commented Jun 12, 2023

At https://github.com/IdentityModel/IdentityModel/blob/0f96912f747c0166bd653ed63d9055c33d5137f5/src/JwtClaimTypes.cs#LL157C21-L157C21 the claim "role" is defined. However, in the IANA JWT Claims registry there is "roles" instead of "role", with references to RFC7643, Section 4.1.2 and RFC9068, Section 2.2.3.1.

There is probably some history behind this, and the standardization of "roles" is a bit recent. But now that there is a standard "roles" claim it would be good to have it in this library along with the other standard claim types. Would it be a way forward to mark the current "role" claim type as deprecated and introduce a new "roles" claim type?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant