fix: Use ClaimTypes.NameIdentifier instead of JwtClaimTypes.Subject.

skoruba / Duende.IdentityServer.Admin

The administration for the Duende IdentityServer and Asp.Net Core Identity ⚡

Apache License 2.0

542 stars 188 forks source link

Open heesung-ich opened 1 month ago

heesung-ich commented 1 month ago

JwtClaimTypes.Subject, used in the GetSubjectId() method, does not get the Id of the logged in user on the current project. Must use ClaimTypes.NameIdentifier to get the Id of the logged in user so that can compare them.

skoruba commented 1 month ago

Hi @heesung-ich - thank you for your PR!

I noticed that tests failed, can you check it please?

thanks.

heesung-ich commented 1 month ago

Hi @heesung-ich - thank you for your PR!

I noticed that tests failed, can you check it please?

thanks.

Modified to use ClaimTypes.NameIdentifier only when a claim is not acquired.