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

[BC Idea]: Get the claims from a SecretText JWT Token #2296

Open
1 task done
jwikman opened this issue Nov 1, 2024 · 2 comments
Open
1 task done

[BC Idea]: Get the claims from a SecretText JWT Token #2296

jwikman opened this issue Nov 1, 2024 · 2 comments
Assignees
Labels
BCIdea Issue related to a BCIdea

Comments

@jwikman
Copy link

jwikman commented Nov 1, 2024

BC Idea Link

https://experience.dynamics.com/ideas/idea/?ideaid=e565077f-7b98-ef11-95f5-7c1e526e605f

Description

When troubleshooting OAuth and trying to figure out why an AccessToken is not being accepted by the resource you are trying to use it on, we have earlier been able to get the AccessToken in plain text and put it into jwt.ms (or any other similar tool) to parse the token and look into all the claims (see https://auth0.com/docs/secure/tokens/json-web-tokens/json-web-token-claims for more information).

Looking at the claims often explains why it does not work, like missing scopes, new configuration not in effect yet (things get cached a lot nowadays), etc.

With the switch to SecretText in the Oath2 module, we cannot get the AccessToken in plain text. Hence, we cannot parse the AccessToken to get the claims. :(

I suggest a new function in the OAuth2 module that has the AccessToken (SecretText) as parameter and returns the claims as a JsonObject.

On top of that we could also add new functions that returns common claims in an easier way, like expiration time (requested in https://experience.dynamics.com/ideas/idea/?ideaid=41f5d251-1a59-ee11-a81c-0022484c1d83).

I will provide the implementation for this BC Idea

  • I will provide the implementation for this BC Idea
@jwikman jwikman added the BCIdea Issue related to a BCIdea label Nov 1, 2024
@JesperSchulz
Copy link
Contributor

@WaelAbuSeada / @darjoo, is this justifiable from a security perspective? Please triage.

@jwikman
Copy link
Author

jwikman commented Nov 6, 2024

@WaelAbuSeada / @darjoo, I can create a draft PR for this if you want to see the implementation approach before making a decision on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BCIdea Issue related to a BCIdea
Projects
None yet
Development

No branches or pull requests

4 participants