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-oidc should start without Keycloak and having to disable the tenant #15599

Closed
sberyozkin opened this issue Mar 10, 2021 · 1 comment · Fixed by #16060
Closed

quarkus-oidc should start without Keycloak and having to disable the tenant #15599

sberyozkin opened this issue Mar 10, 2021 · 1 comment · Fixed by #16060
Labels
area/oidc kind/enhancement New feature or request
Milestone

Comments

@sberyozkin
Copy link
Member

Description
Right now quarkus-oidc can start without Keycloak (other OIDC provider) if quarkus.oidc.tenant-enabled=false for the default configuration/tenant. This poses a problem for the dev mode in particular. It is not consistent with the fact that the dynamic tenants can be easily allocated at runtime with TenantConfigResolver with Keycloak not even being available at the start up.
quarkus.oidc.tenant-enabled can remain to be useful on its own - but quarkus-oidc should be able to start even if the default tenant is enabled and Keycloak is not available - it has been proposed by some users before.

Implementation ideas
OidcRecorder should avoid resolving Uni for the default and other statically configured tenants and delay it till the tenant is requested at runtime

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/oidc kind/enhancement New feature or request
Projects
None yet
2 participants