-
Notifications
You must be signed in to change notification settings - Fork 359
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: oauth intsropsection configurable timeout #370
Conversation
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.
This looks pretty good, thank you! I have some ideas how to improve it.
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.
Thank you!
hmm, I was trying to use this feature and it seems like this only works if pre_authorization is enabled? |
Yup, that's not correct - would you be open to fix this? :) |
sure, can do :) |
Related issue
#310
Proposed changes
Make http timeouts configurable in the oauth2 introspector
Checklist
vulnerability, I confirm that I got green light (please contact [email protected]) from the maintainers to push the changes.