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

Update to JWTKit 5 #14

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Update to JWTKit 5 #14

wants to merge 8 commits into from

Conversation

0xTim
Copy link
Contributor

@0xTim 0xTim commented Nov 9, 2023

Update library to use the new JWTKit 5

Builds on top of #13 so that should be merged first

@0xTim
Copy link
Contributor Author

0xTim commented Nov 9, 2023

cc @ptoffy

@alexanderjordanbaker
Copy link
Collaborator

@0xTim Are you still planning to move this PR from Draft -> Ready?

@0xTim
Copy link
Contributor Author

0xTim commented Jan 30, 2024

@alexanderjordanbaker yes we will do once we have a release of the new version of JWTKit with a stable API

@0xTim
Copy link
Contributor Author

0xTim commented Feb 27, 2024

@alexanderjordanbaker this is pretty much ready. I don't know whether you want to wait until JWTKit v5 is out of Beta or you're happy to take it as is. We're not anticipating any API changes

@0xTim
Copy link
Contributor Author

0xTim commented Mar 14, 2024

@ptoffy FYI some conflicts here (also we should have a discussion about the Sendable stuff)

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

Successfully merging this pull request may close these issues.

None yet

4 participants