Added link to Swagger UI from OIDC Dev UI Screen #20005
Conversation
Signed-off-by:Phillip Kruger <[email protected]>
quarkus-bot
bot
added
area/dependencies
| // In dev mode, default to persist Authorization true | ||
| if (!swaggerUiConfig.persistAuthorization.isPresent()) { | ||
| swaggerUiConfig.persistAuthorization = Optional.of(true); | ||
| } |
There was a problem hiding this comment.
Instead of making it an Optional, isn't easier to define as a boolean with true as the default value?
There was a problem hiding this comment.
The reason for this is that there is actually no default. In dev mode we make it true, except if explicitly set false by the user. In other modes it will be false. except if explicitly set true by the user
There was a problem hiding this comment.
My only concern is that it's not common to change config values in a processor, so perhaps this could be tested when this value is consumed instead
There was a problem hiding this comment.
It gets consumed in SmallRye, and there we do not have the concept of dev mode...
There was a problem hiding this comment.
@phillip-kruger Very nice, thanks for making it happen, look forward to documenting it :-)
phillip-kruger
added
the
triage/waiting-for-ci
|
Should this be backported? |
quarkus-bot
bot
removed
the
triage/waiting-for-ci
This PR allows you to navigate from the OIDC DEV UI screen to Swagger (if included) in a logged in state.
Signed-off-by:Phillip Kruger [email protected]