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

feat(client,js)!: support custom jwt verifier #622

Merged
merged 4 commits into from
Feb 19, 2024
Merged

Conversation

gao-sun
Copy link
Member

@gao-sun gao-sun commented Feb 15, 2024

Summary

implement changes to make they work in React Native. see changesets for notable changes. in addition:

  • remove homemade cached jwk set as the new jwt verifier can hold a instance of createRemoteJWKSet, which already has the cache mechanism

Testing

  • updated test cases accordingly
  • tested with the React Native sample, sign-in and sign-out works as expected

Checklist

  • .changeset
  • unit tests
  • integration tests
  • necessary TSDoc comments

Sorry, something went wrong.

Verified

This commit was signed with the committer’s verified signature.
gao-sun Gao Sun
@gao-sun gao-sun force-pushed the gao-refactor-client branch from bba0cda to 7d6b606 Compare February 15, 2024 06:20

Verified

This commit was signed with the committer’s verified signature.
gao-sun Gao Sun
@gao-sun gao-sun force-pushed the gao-refactor-client branch 2 times, most recently from 3c42ded to 46b4ed1 Compare February 17, 2024 09:47

Verified

This commit was signed with the committer’s verified signature.
gao-sun Gao Sun
@gao-sun gao-sun force-pushed the gao-refactor-client branch from 46b4ed1 to 892d9cf Compare February 17, 2024 10:10

Verified

This commit was signed with the committer’s verified signature.
gao-sun Gao Sun
@gao-sun gao-sun marked this pull request as ready for review February 17, 2024 15:42
@gao-sun gao-sun merged commit 012b7c4 into master Feb 19, 2024
22 checks passed
@gao-sun gao-sun deleted the gao-refactor-client branch February 19, 2024 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants