Adds TokenCredential PrincipalId lookup

Azure / Microsoft.Azure.StackExchangeRedis

Azure-specific wrapper for the StackExchange.Redis client library

MIT License

17 stars 14 forks source link

Adds TokenCredential PrincipalId lookup #43

Closed PascalSenn closed 6 months ago

PascalSenn commented 9 months ago

Adds a lookup for the principal id

PascalSenn commented 9 months ago

@microsoft-github-policy-service agree

sabbadino commented 9 months ago

Why not use jwttokenhandler ? https://learn.microsoft.com/en-us/dotnet/api/system.identitymodel.tokens.jwt.jwtsecuritytokenhandler?view=msal-web-dotnet-latest

PascalSenn commented 9 months ago

@sabbadino Microsoft.Azure.StackExchangeRedis does not have a dependency on System.IdentityModel at the moment.

arealmaas commented 8 months ago

omg looking forward to this 🥰

philon-msft commented 6 months ago

@PascalSenn thank you for putting together this PR! I ended up extracting the oid a little differently in #48 with a new dependency on Microsoft.IdentityModel.JsonWebTokens, so I'll close this PR. Let me know if you have any further suggestions.