Skip to content

fix(helm): let the backend fetch Keycloak keys from inside the cluster - #257

Merged
cmdoret merged 2 commits into
mainfrom
fix/deployments
Aug 27, 2026
Merged

fix(helm): let the backend fetch Keycloak keys from inside the cluster#257
cmdoret merged 2 commits into
mainfrom
fix/deployments

Conversation

@sabinem

@sabinem sabinem commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

The backend looked up its login keys at the public address (auth.), which only works once DNS, the ingress and the TLS certificate all exist. On a fresh cluster none of them do, so the lookup failed and every logged-in request was rejected — while the pod still looked healthy.

It now asks Keycloak directly, pod to pod. The issuer url stays public, because that is the address written inside the tokens.

Fails at install time with a clear message if Keycloak is not part of the release, since then there is no pod to ask.

The backend looked up its login keys at the public address
(auth.<domain>), which only works once DNS, the ingress and the TLS
certificate all exist. On a fresh cluster none of them do, so the
lookup failed and every logged-in request was rejected — while the
pod still looked healthy.

It now asks Keycloak directly, pod to pod. The issuer url stays
public, because that is the address written inside the tokens.

Fails at install time with a clear message if Keycloak is not part
of the release, since then there is no pod to ask.
@sabinem
sabinem marked this pull request as draft August 27, 2026 05:00
@cmdoret
cmdoret marked this pull request as ready for review August 27, 2026 07:20
@sabinem sabinem closed this Aug 27, 2026
@sabinem sabinem reopened this Aug 27, 2026
@cmdoret
cmdoret merged commit 79e2e06 into main Aug 27, 2026
2 of 4 checks passed
@cmdoret
cmdoret deleted the fix/deployments branch August 27, 2026 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants