From f6d7510a144aeb0084a16f0afac04847c0b662dd Mon Sep 17 00:00:00 2001
From: Alon Girmonsky <1990761+alongir@users.noreply.github.com>
Date: Tue, 23 Jan 2024 21:30:24 -0800
Subject: [PATCH] fix the env variable / helm issue (#1486)

* fix the env variable / helm issue

Empty environment variables can not be read by front.

* change env variable to avoid an empty string
---
 helm-chart/templates/06-front-deployment.yaml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/helm-chart/templates/06-front-deployment.yaml b/helm-chart/templates/06-front-deployment.yaml
index b2c109373..721a09afc 100644
--- a/helm-chart/templates/06-front-deployment.yaml
+++ b/helm-chart/templates/06-front-deployment.yaml
@@ -29,9 +29,9 @@ spec:
             - name: REACT_APP_AUTH_ENABLED
               value: '{{ .Values.tap.auth.enabled }}'
             - name: REACT_APP_AUTH_TYPE
-              value: '{{ .Values.tap.auth.type }}'
+              value: '{{ not (eq .Values.tap.auth.type "") | ternary .Values.tap.auth.type " " }}'
             - name: REACT_APP_AUTH_SAML_IDP_METADATA_URL
-              value: '{{ .Values.tap.auth.saml.idpMetadataUrl }}'
+              value: '{{ not (eq .Values.tap.auth.saml.idpMetadataUrl "") | ternary .Values.tap.auth.saml.idpMetadataUrl " " }}'
             - name: REACT_APP_REPLAY_DISABLED
               value: '{{ .Values.tap.replayDisabled }}'
           image: '{{ .Values.tap.docker.registry }}/front:{{ not (eq .Values.tap.docker.tag "") | ternary .Values.tap.docker.tag (printf "v%s" .Chart.Version) }}'