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

Remove StandardClaims in favor of RegisteredClaims #235

Merged
merged 1 commit into from Aug 28, 2022

Conversation

oxisto
Copy link
Collaborator

@oxisto oxisto commented Aug 27, 2022

This PR removes the old legacy standard claims, which have been deprecated since the beginning of the v4 module in favor of the newer RegisteredClaims. Removing them before any further changes to the validation API is quite useful, as less code needs to be adapated.

This PR removes the old legacy standard claims, which have been deprecated since the beginning of the `v4` module in favor of the newer `RegisteredClaims`. Removing them before any further changes to the validation API is quite useful, as less code needs to be adapated.
@oxisto oxisto changed the base branch from main to v5 August 27, 2022 10:08
@oxisto oxisto changed the title remove standard claims Remove StandardClaims in favor of RegisteredClaims Aug 27, 2022
@oxisto oxisto requested review from mfridman, lggomez and Waterdrips and removed request for mfridman and lggomez August 27, 2022 10:08
@oxisto oxisto added the next The next iteration of development, currently `v6` label Aug 27, 2022
@oxisto oxisto merged commit 7e82f33 into v5 Aug 28, 2022
@oxisto oxisto deleted the remove-standard-claims branch August 28, 2022 16:17
oxisto added a commit that referenced this pull request Dec 9, 2022
This PR removes the old legacy standard claims, which have been deprecated since the beginning of the `v4` module in favor of the newer `RegisteredClaims`. Removing them before any further changes to the validation API is quite useful, as less code needs to be adapated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next The next iteration of development, currently `v6`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant