improvement: generate default oauth2RedirectUrl based on page location #7476
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.
… including the path (an omission of #5085)
Description
The previous improvement (#5085) assumed that the Swagger UI was hosted at the root path.
Motivation and Context
Though this might be true for some situation, it's likely to be false for many deployments. By involving the path into the oauth2RedirectUrl, the Oauth2 flow would typically work more often.
How Has This Been Tested?
I initally wrote a rewriting filter that applied the fix to a running deployment of Spring Boot. When that worked, I applied the same fix to the code, and wrote 3 additional tests that prove the fix.
Screenshots (if appropriate):
Checklist
My PR contains...
src/
is unmodified: changes to documentation, CI, metadata, etc.)package.json
)My changes...
Documentation
Automated tests