-
Notifications
You must be signed in to change notification settings - Fork 17
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: Add user defined domain validation #227
Conversation
…owed domains list" This reverts commit 4d484ce.
No linked issues found. Please add the corresponding issues in the pull request description. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@CyanoFresh can you check if this would solve your issue?
Co-authored-by: Simone Busoli <simone.busoli@gmail.com>
yes! |
I agree, this is probably a leftover from the former implementation which was exclusively auth0. @eugenio-oddone can you create an issue to track this request? |
…com/nearform/get-jwks into update-allowed-domains-verification
Yes will do 👍 |
Relates to #226
Adds support for an optional function to validate a token's domain when allowedDomains functionality does not suffice.