Skip to content

Navigation Menu

Sign in
Sign up

Token Verifier Improvements - #1147

Draft
javiertoledo wants to merge 11 commits into
main from
feature/optional-tokenverifier-issuer
Draft

Token Verifier Improvements #1147
javiertoledo wants to merge 11 commits into
main from
feature/optional-tokenverifier-issuer

Conversation

@javiertoledo

@javiertoledo javiertoledo commented Aug 2, 2022
edited
Loading

Copy link
Copy Markdown
Member

Rework #1111 to adapt the changes to the new TokenVerifier interface and class structures.

  • Replaces jsonwebtoken and jwks-rsa dependencies with jose because jose has zero dependencies.
  • Adds a new EncryptedTokenVerifier token verifier implementation which can be used to decrypt JWE tokens. This can be useful when using libraries like NextAuth.js.
  • Given the higher level of abstraction of jose, the JWT utility methods are no longer needed and have been removed.

Pending before closing this PR:

  • Fix tests and verify they're all passing
  • Update the documentation with the affected changes (removal of the JWT utilities and creation of the new EncryptedTokenVerifier class).

thomas-advantitge reacted with thumbs up emoji
Javier Toledo added 2 commits August 5, 2022 16:21
...nverifier-issuer
# Conflicts:
#	package-lock.json
#	packages/framework-core/src/booster-token-verifier.ts
#	packages/framework-types/src/concepts/token-verifier-config.ts
javiertoledo force-pushed the feature/optional-tokenverifier-issuer branch from 180a304 to 3de75ec Compare August 5, 2022 17:13
Javier Toledo
WIP
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /