file: prefix is not working for static jwks configurations
#1635
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
jbreton
changed the title
file: prefix is not working for static jwks configurations
|
Here is the issue for the ReadTimeout #1633 |
sdelamo
requested changes
Mar 14, 2024
|
@sdelamo It's my first time working with groovy. I looked at other tests to see what I could do. Those tests are far from perfect but it's better than nothing at all. |
sdelamo
approved these changes
Mar 18, 2024
sdelamo
pushed a commit
that referenced
this pull request
Mar 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix for #1634
We found this bug while setting a static Jwks as a workaround of a possible bug regarding a ReadTimeout that could occur when fetching a Jwks from an url.
We are still investigating the cause of the ReadTimeout and haven't opened an issue since it could be caused by our custom JWT validator. Maybe this will ring a bell as it began when we updated to micronaut 4.3.1 and is still occuring in 4.3.4.