Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Remove the need for Token.js #13

Closed
darahayes opened this issue Jul 10, 2019 · 0 comments
Closed

Remove the need for Token.js #13

darahayes opened this issue Jul 10, 2019 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@darahayes
Copy link
Contributor

src/KeycloakToken.ts is a direct copy of the Token class within the keycloak-nodejs-connect library. We copied it because the Keycloak library does not expose that class directly.

However, I have found keycloak.grantManager.createGrant is the more correct way for us to parse and validate the token. If we use this inside the KeycloakSubscriptionHandler, then we can completely remove the Token class.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant