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

quarkus.swagger-ui.oauth2-redirect-url documentation and default value #19113

Closed
tttomat19 opened this issue Jul 30, 2021 · 2 comments · Fixed by #19148
Closed

quarkus.swagger-ui.oauth2-redirect-url documentation and default value #19113

tttomat19 opened this issue Jul 30, 2021 · 2 comments · Fixed by #19148
Assignees
Labels
Milestone

Comments

@tttomat19
Copy link
Contributor

Description

So I battled swagger-ui + oidc with auth code or implicit flow with redirects and such.
The problem was after successful login in oauth provider the redirect led to localhost:8080/oauth2-redirect.html
And there was 404.

The solution was found here in the last comment: #4766 Many thanks to Phillip.

I set this property and it worked fine.
quarkus.swagger-ui.oauth2-redirect-url=/q/swagger-ui/oauth2-redirect.html

Implementation ideas

I believe the setting quarkus.swagger-ui.oauth2-redirect-url should be set to /q/swagger-ui/oauth2-redirect.html by default. Or maybe if I'm wrong it might be good to add some more explanation to the property meaning in docs.

@tttomat19 tttomat19 added the area/housekeeping Issue type for generalized tasks not related to bugs or enhancements label Jul 30, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Jul 30, 2021

/cc @MikeEdgar, @phillip-kruger

@phillip-kruger
Copy link
Member

Hi @tttomat19 - Thanks for logging this. I'll play around with making that the default. As a last resort, I'll update the documentation

@phillip-kruger phillip-kruger self-assigned this Jul 30, 2021
@gastaldi gastaldi added kind/enhancement New feature or request and removed area/housekeeping Issue type for generalized tasks not related to bugs or enhancements labels Jul 30, 2021
@quarkus-bot quarkus-bot bot added this to the 2.2 - main milestone Aug 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants