Skip to content

Commit d62a898

Browse files
authored
Merge pull request #4433 from dihmandrake/main
charts: Quote OIDC_USE_PKCE value to ensure it's a string
2 parents d9a3326 + b4fe997 commit d62a898

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

charts/headlamp/templates/deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ spec:
216216
{{- end }}
217217
{{- if $oidc.usePKCE }}
218218
- name: OIDC_USE_PKCE
219-
value: {{ $oidc.usePKCE }}
219+
value: {{ $oidc.usePKCE | quote }}
220220
{{- end }}
221221
{{- if $oidc.meUserInfoURL }}
222222
- name: ME_USER_INFO_URL

charts/headlamp/tests/expected_templates/oidc-pkce.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ spec:
107107
- name: OIDC_SCOPES
108108
value: testScope
109109
- name: OIDC_USE_PKCE
110-
value: true
110+
value: "true"
111111
args:
112112
- "-in-cluster"
113113
- "-plugins-dir=/headlamp/plugins"

0 commit comments

Comments
 (0)