Merge pull request #32 from DataKitchen/release/2.7.4

Release/2.7.4

Author: datakitchen-devops

deploy/charts/observability-app/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: dataops-observability-app
 type: application
 appVersion: "2.x.x"
-version: "2.3.0"
+version: "2.3.1"
 
 description: DataOps Observability
 home: https://datakitchen.io

deploy/charts/observability-app/templates/observability-ui.yaml

@@ -62,7 +62,7 @@ spec:
             - name: OBSERVABILITY_API_HOSTNAME
               value: {{ tpl .Values.observability_api.hostname . | quote }}
             {{- end }}
-            {{- if .Values.observability_ui.csp_extra }}
+            {{- if not (quote .Values.observability_ui.csp_extra | empty) }}
             - name: OBSERVABILITY_CSP_EXTRA
               value: {{ tpl .Values.observability_ui.csp_extra . | quote }}
             {{- end }}

observability_ui/libs/core/src/lib/services/auth/session.service.spec.ts

@@ -80,7 +80,7 @@ describe('Session Service', () => {
     it('should set the token into the cookies', () => {
       const token = 'my-test-token';
       sessionService.setToken(token);
-      expect(cookieService.set).toBeCalledWith(cookieKeys.token, token, { path: cookiePath, secure: true });
+      expect(cookieService.set).toBeCalledWith(cookieKeys.token, token, { path: cookiePath });
     });
   });
 

observability_ui/libs/core/src/lib/services/auth/session.service.ts

@@ -49,7 +49,7 @@ export class SessionService {
   }
 
   setToken(token: string): void {
-    this.cookieService.set(cookieKeys.token, token, { path: cookiePath, secure: true });
+    this.cookieService.set(cookieKeys.token, token, { path: cookiePath });
   }
 
   removeToken(): void {