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(contrib-jwt): Add parameters to set the JWT extras field #2313

Merged
merged 3 commits into from
Sep 18, 2023

Conversation

dialvarezs
Copy link
Contributor

@dialvarezs dialvarezs commented Sep 18, 2023

Pull Request Checklist

  • New code has 100% test coverage
  • (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
  • (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR

Description

This PR adds token_extras to both login() and create_token() methods, to allow the definition of the extras JWT field.

- Add `token_extras` to both `login()` and `create_token()` methods.
- Update tests and docs example to use the new parameter.
@dialvarezs dialvarezs requested review from a team as code owners September 18, 2023 07:36
Copy link
Member

@cofin cofin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cofin cofin enabled auto-merge (squash) September 18, 2023 18:03
@cofin cofin merged commit 30f748e into litestar-org:main Sep 18, 2023
15 checks passed
@github-actions
Copy link

Documentation preview will be available shortly at https://litestar-org.github.io/litestar-docs-preview/2313

@cofin
Copy link
Member

cofin commented Sep 18, 2023

@all-contributors add @dialvarezs for code, test, and docs

@allcontributors
Copy link
Contributor

@cofin

I've put up a pull request to add @dialvarezs! 🎉

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.

2 participants