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.

Sign up for GitHub

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

Jump to bottom

Refactor SimpleJWT\Crypt namespace, JWT and JWE #60

Merged
merged 7 commits into from
Mar 6, 2022
Merged

Refactor SimpleJWT\Crypt namespace, JWT and JWE #60

merged 7 commits into from
Mar 6, 2022

Conversation

kelvinmo
Copy link
Owner

@kelvinmo kelvinmo commented Feb 18, 2022 •
edited
Loading

Split files in SimpleJWT\Crypt into three separate namespaces:

  1. SimpleJWT\Crypt\Signature
  2. SimpleJWT\Crypt\Encryption
  3. SimpleJWT\Crypt\KeyManagement

@kelvinmo kelvinmo added this to the v0.7 milestone Feb 18, 2022
@kelvinmo kelvinmo changed the title Refactor SimpleJWT\Crypt namespace Refactor SimpleJWT\Crypt namespace and update JWT and JWE to derive from a single class Feb 19, 2022
@kelvinmo
Copy link
Owner Author

This pull request also refactors JWT and JWE to derive from a single class Token.

@kelvinmo kelvinmo changed the title Refactor SimpleJWT\Crypt namespace and update JWT and JWE to derive from a single class Refactor SimpleJWT\Crypt namespace, JWT and JWE Feb 19, 2022
@kelvinmo kelvinmo merged commit d3dc0b2 into master Mar 6, 2022
@kelvinmo kelvinmo deleted the features/namespace-refactor branch March 6, 2022 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.7
Development

Successfully merging this pull request may close these issues.
1 participant
@kelvinmo