Redis session fix (#33)

* Redis fixes, remove legacy option

* Version bump for fix

.devcontainer/devcontainer.json

@@ -10,13 +10,12 @@
         "esbenp.prettier-vscode",
         "mikestead.dotenv",
         "ms-azuretools.vscode-bicep",
-        "github.vscode-pull-request-github"
+        "github.vscode-pull-request-github",
+        "anweber.vscode-httpyac"
       ]
     }
   },
 
-  // Optional features, uncomment to enable.
-  // See https://code.visualstudio.com/docs/remote/containers#_dev-container-features-preview
   "features": {
     "azure-cli": "latest"
   },

build/Dockerfile

@@ -2,7 +2,7 @@ ARG ARCH=
 ARG IMAGE_BASE=20-alpine
 
 FROM ${ARCH}node:$IMAGE_BASE
-LABEL Name="Node.js Demo App" Version=4.9.8
+LABEL Name="Node.js Demo App" Version=4.9.9
 LABEL org.opencontainers.image.source = "https://github.com/benc-uk/nodejs-demoapp"
 ENV NODE_ENV production
 WORKDIR /app 

deploy/kubernetes/redis.yaml

@@ -30,7 +30,7 @@ spec:
     spec:
       containers:
         - name: redis
-          image: redis:6
+          image: redis
           ports:
             - containerPort: 6379
           resources:

src/package.json

@@ -1,7 +1,7 @@
 {
   "name": "nodejs-demoapp",
   "description": "Node.js Express app for demos",
-  "version": "4.9.8",
+  "version": "4.9.9",
   "author": "Ben Coleman",
   "engines": {
     "node": ">=20.0.0"

src/server.mjs

@@ -52,7 +52,7 @@ const sessionConfig = {
 
 // Very optional Redis session store - only really needed when running multiple instances
 if (process.env.REDIS_SESSION_HOST) {
-  const redisClient = createRedisClient({ legacyMode: true, url: `redis://${process.env.REDIS_SESSION_HOST}` })
+  const redisClient = createRedisClient({ url: `redis://${process.env.REDIS_SESSION_HOST}` })
 
   redisClient.connect().catch((err) => {
     console.error('### 🚨 Redis session store error:', err.message)