DTSCCI-3814: code clean up

Author: “elvianihuseyin2”

charts/cmc-citizen-frontend/values.preview.template.yaml

@@ -19,9 +19,8 @@ nodejs:
     targetCPUUtilizationPercentage: 80 # Default is 80% target CPU utilization
   environment:
     REFORM_TEAM: cmc
-    REFORM_SERVICE_NAME: citizen-frontend
-    # Use Redis (civil-citizen-ui-draft-store) in preview; keep a non-production session secret for cookie signing
-    SESSION_SECRET: "preview-session-secret-not-for-production-min-32-chars"
+    REFORM_SERVICE_NAME: hmctspublic.azurecr.io/cmc/citizen-frontend:latest
+    SESSION_SECRET: "preview-session-secret"
     LOG_LEVEL: DEBUG
     HTTP_TIMEOUT: 60000
     FEATURE_TOGGLES_API_URL: http://${SERVICE_NAME}-ftr-tgl

config/default.json

@@ -32,7 +32,7 @@
   },
   "session": {
     "cookieName": "cmc-citizen-session",
-    "secret": "not-a-real-session-secret-min-32-chars",
+    "secret": "local-session-secret",
     "useRedisStore": false,
     "maxAgeInMinutes": 120,
     "redis": {

src/main/server.ts

@@ -6,6 +6,7 @@ import { Logger } from '@hmcts/nodejs-logging'
 
 const logger = Logger.getLogger('server.ts')
 
+// App Insights needs to be enabled as early as possible as it monitors other libraries as well
 new AppInsights().enable()
 logger.info('enabling App Insights')