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

JWT tid claim transformation to "http://schemas.microsoft.com/identity/claims/tenantid" #685

Closed
Tratcher opened this issue Aug 22, 2017 · 2 comments
Assignees
Labels
Customer reported Indicates issue was opened by customer P1 More important, prioritize highly
Milestone

Comments

@Tratcher
Copy link
Contributor

From @cartervms on August 16, 2017 19:46

Is there any way to disable this conversion? This is causing issues for us when requests flow between different services, where some are using ASP.NET core and others aren't. This is really inconvenient, because now our shared claim lookup code is having to do .Where( c => c.Equals("http://schemas.microsoft.com/identity/claims/tenantid") || c.Equals("tid"));

Copied from original issue: aspnet/Security#1372

@Tratcher
Copy link
Contributor Author

From @cartervms on August 16, 2017 19:46

For reference: https://docs.microsoft.com/en-us/azure/architecture/multitenant-identity/claims

This table lists the claim types as they appear in the ID token. In ASP.NET Core, the OpenID Connect middleware converts some of the claim types when it populates the Claims collection for the user principal:
oid > http://schemas.microsoft.com/identity/claims/objectidentifier
tid > http://schemas.microsoft.com/identity/claims/tenantid
unique_name > http://schemas.xmlsoap.org/ws/2005/05/identity/claims/name
upn > http://schemas.xmlsoap.org/ws/2005/05/identity/claims/upn

@brentschmaltz brentschmaltz added the P1 More important, prioritize highly label Aug 22, 2017
@brentschmaltz brentschmaltz self-assigned this Aug 22, 2017
@brentschmaltz brentschmaltz added this to the 5.2.0 milestone Aug 22, 2017
@brentschmaltz brentschmaltz added the Customer reported Indicates issue was opened by customer label Aug 22, 2017
@brentschmaltz
Copy link
Member

@cartervms @Tratcher
see: #550
We will handle this post preview.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer reported Indicates issue was opened by customer P1 More important, prioritize highly
Projects
None yet
Development

No branches or pull requests

2 participants