fix: make customHeadersConfigMap optional (#112)

Signed-off-by: Lester Guerzon <[email protected]>
guerzon · Aug 30, 2024 · bbafc1e · bbafc1e
1 parent 6c1cf6a
commit bbafc1e
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/charts/vaultwarden/Chart.yaml b/charts/vaultwarden/Chart.yaml
@@ -13,5 +13,5 @@ maintainers:
   - name: guerzon
     email: [email protected]
     url: https://github.com/guerzon
-version: 0.25.0
+version: 0.25.1
 kubeVersion: ">=1.12.0-0"
diff --git a/charts/vaultwarden/templates/configmap.yaml b/charts/vaultwarden/templates/configmap.yaml
@@ -1,10 +1,12 @@
+{{- with .Values.ingress.customHeadersConfigMap }}
 apiVersion: v1
 kind: ConfigMap
 metadata:
   name: custom-headers-configmap
 data:
-  Request-Id: $req_id
+  {{- toYaml . | nindent 2 }}
 ---
+{{- end }}
 apiVersion: v1
 kind: ConfigMap
 metadata:

diff --git a/charts/vaultwarden/values.yaml b/charts/vaultwarden/values.yaml
@@ -697,9 +697,10 @@ ingress:
   ##
   nginxAllowList: ""
   ## @param ingress.customHeadersConfigMap ConfigMap containing custom headers to be added to the ingress.
-  ## Defaults to "custom-headers-configmap", which sets a Request-ID header.
+  ## Requirement: First define the allowed response headers in global-allowed-response-headers.
+  ## Ref: https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/annotations/#custom-headers
   ##
-  customHeadersConfigMap: "custom-headers-configmap"
+  customHeadersConfigMap: {}
   ## TODO:
   ##   - Add support for using cert-manager.
   ##   - Support for multiple TLS hostnames.