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

Add optionalToken parameter to middleware #342

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

magnusburton
Copy link

Description

I thought about adding a parameter to allow for tokens to be read, while still allowing tokens to be retired, expired, invalid, or non existent without anything being thrown. If a valid token can be decoded, it'll be returned. The PR does not break anything as it's false by default, so existing implementations will still work.

References

I figured I'd do this due to my own personal need, as well as a couple of discussions within this repository, #194

Testing

I've not yet got around to add tests. If someone would like to contribute with some, please do!

  • This change adds test coverage for new/changed/fixed functionality

Checklist

  • I have added documentation for new/changed functionality in this PR or in auth0.com/docs
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used, if not the default branch

@magnusburton magnusburton marked this pull request as draft April 11, 2024 19:30
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.

1 participant