Skip to content

Commit f2385b7

Browse files
committed
fix(helm): disable Keycloak test user by default
Production deployments must not ship a known dev/dev credential. The test user is now opt-in via values-local.yaml (cluster:install) and values-test.yaml (e2e tests). Assisted-By: Claude (Anthropic AI) <noreply@anthropic.com> Signed-off-by: Radek Ježek <radek.jezek@ibm.com>
1 parent 18ccc62 commit f2385b7

5 files changed

Lines changed: 22 additions & 3 deletions

File tree

deploy/helm/humr/templates/NOTES.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,9 @@ Components:
1414
- Keycloak: {{ include "humr.keycloak.fullname" . }}.{{ .Release.Namespace }}.svc:{{ .Values.keycloak.port }}
1515
- Admin: {{ include "humr.url.keycloak" . }} (user: {{ .Values.keycloak.admin.user }})
1616
- Realm: {{ .Values.keycloak.realm }}
17+
{{- if .Values.keycloak.testUser.enabled }}
1718
- Test user: {{ .Values.keycloak.testUser.username }} / {{ .Values.keycloak.testUser.password }}
19+
{{- end }}
1820
{{- end }}
1921

2022
Agent namespace: {{ .Values.agentNamespace }}

deploy/helm/humr/templates/keycloak/realm-configmap.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -137,6 +137,7 @@ data:
137137
}
138138
],
139139
"users": [
140+
{{- if .Values.keycloak.testUser.enabled }}
140141
{
141142
"username": {{ .Values.keycloak.testUser.username | quote }},
142143
"enabled": true,
@@ -152,6 +153,7 @@ data:
152153
}
153154
]
154155
}
156+
{{- end }}
155157
]
156158
}
157159
{{- end }}

deploy/helm/humr/values-local.yaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,3 +24,10 @@ defaultTemplate:
2424
repository: humr-example-agent
2525
tag: latest
2626
pullPolicy: Never
27+
28+
# Bootstrap a known dev/dev user for local cluster — never enabled in production.
29+
keycloak:
30+
testUser:
31+
enabled: true
32+
username: dev
33+
password: dev

deploy/helm/humr/values-test.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,11 @@ onecli:
1919
memory: 512Mi
2020

2121
keycloak:
22+
# E2E tests log in as dev/dev — never enabled in production.
23+
testUser:
24+
enabled: true
25+
username: dev
26+
password: dev
2227
resources:
2328
requests:
2429
cpu: 100m

deploy/helm/humr/values.yaml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -126,10 +126,13 @@ keycloak:
126126
controllerClientId: humr-controller
127127
controllerClientSecret: ""
128128

129-
# -- Test user bootstrapped via realm import
129+
# -- Test user bootstrapped via realm import.
130+
# DISABLED by default — production deployments must not ship a known credential.
131+
# Local dev (values-local.yaml) and e2e tests (values-test.yaml) enable it.
130132
testUser:
131-
username: dev
132-
password: dev
133+
enabled: false
134+
username: ""
135+
password: ""
133136

134137
# -- Database connection (empty host = use shared local postgres)
135138
db:

0 commit comments

Comments
 (0)