[bitnami/keycloak]: add url env vars to fix admin ingress access (#25386)

* [bitnami/keycloak]: add url env vars to fix admin ingress access

Replace KC_HOSTNAME_ADMIN by KC_HOSTNAME_ADMIN_URL to include the scheme
(http or https) and add KC_HOSTNAME_URL for the frontend to function
correctly. These variable allow the front to function correctly if we
have the tls option enabled. Before this commit the front was always
attempting to load some resources over http while it may be exposed in
https.

Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@ledger.fr>

* [bitnami/keycloak]: fix license header to fix the CI

Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@ledger.fr>

---------

Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@ledger.fr>
This commit is contained in:
Arthur Outhenin-Chalandre
2024-04-29 15:39:35 +02:00
committed by GitHub
parent ec7d04e139
commit 43ccdb2e90
2 changed files with 8 additions and 4 deletions

View File

@@ -1,4 +1,4 @@
# Copyright VMware, Inc.
# Copyright Broadcom, Inc. All Rights Reserved.
# SPDX-License-Identifier: APACHE-2.0
annotations:
@@ -33,4 +33,4 @@ maintainers:
name: keycloak
sources:
- https://github.com/bitnami/charts/tree/main/bitnami/keycloak
version: 21.0.4
version: 21.1.0

View File

@@ -213,8 +213,12 @@ spec:
value: {{ .Values.extraStartupArgs | quote }}
{{- end }}
{{- if .Values.adminIngress.enabled }}
- name: KC_HOSTNAME_ADMIN
value: {{ include "common.tplvalues.render" (dict "value" .Values.adminIngress.hostname "context" $) }}
- name: KC_HOSTNAME_ADMIN_URL
value: "http{{ if .Values.adminIngress.tls }}s{{ end }}://{{ include "common.tplvalues.render" (dict "value" .Values.adminIngress.hostname "context" $) }}"
{{- end }}
{{- if and .Values.adminIngress.enabled (not .Values.ingress.enabled) }}
- name: KC_HOSTNAME_URL
value: "http{{ if .Values.adminIngress.tls }}s{{ end }}://{{ include "common.tplvalues.render" (dict "value" .Values.adminIngress.hostname "context" $) }}"
{{- end }}
{{- if .Values.extraEnvVars }}
{{- include "common.tplvalues.render" (dict "value" .Values.extraEnvVars "context" $) | nindent 12 }}