feat: make Redis session store optional

Author: Ihor Masechko

docker-compose.yml

@@ -37,7 +37,6 @@ services:
     ports:
       - "3000:3000"
     environment:
-      - REDIS_URI=redis://redis:6379
       - BASE_URL=http://localhost:3000
       - NODE_ENV=development
       - APOS_MONGODB_URI=mongodb://mongodb:27017/apostrophe
@@ -61,7 +60,6 @@ services:
     depends_on:
       - mongodb
       - localstack
-      - redis
     restart: unless-stopped
     healthcheck:
       test:
@@ -118,25 +116,6 @@ services:
     networks:
       - proxynet
 
-  # Redis for caching (optional, but recommended for production)
-  redis:
-    image: redis:7-alpine
-    container_name: apostrophe-redis
-    ports:
-      - "6379:6379"
-    volumes:
-      - redis_data:/data
-    command: redis-server --appendonly yes
-    restart: unless-stopped
-    healthcheck:
-      test: ["CMD", "redis-cli", "ping"]
-      interval: 10s
-      timeout: 5s
-      retries: 3
-    networks:
-      - proxynet
-
 volumes:
   mongodb_data:
-  redis_data:
   localstack_data:

website/app.js

@@ -1,8 +1,23 @@
 const apostrophe = require('apostrophe');
 require('dotenv').config({ path: '../.env' });
-const { getEnv } = require('./utils/env');
+const { getEnv, getOptionalEnv } = require('./utils/env');
 
 function createAposConfig() {
+  const redisUri = getOptionalEnv('REDIS_URI');
+
+  const sessionConfig = {
+    secret: getEnv('SESSION_SECRET'),
+  };
+
+  if (redisUri) {
+    sessionConfig.store = {
+      connect: require('connect-redis'),
+      options: {
+        url: redisUri,
+      },
+    };
+  }
+
   return {
     shortName: 'apostrophe-site',
     baseUrl: process.env.BASE_URL || 'https://speedandfunction.com',
@@ -13,16 +28,7 @@ function createAposConfig() {
       '@apostrophecms/security-headers': {},
       '@apostrophecms/express': {
         options: {
-          session: {
-            // If using Redis (recommended for production)
-            secret: getEnv('SESSION_SECRET'),
-            store: {
-              connect: require('connect-redis'),
-              options: {
-                url: getEnv('REDIS_URI'),
-              },
-            },
-          },
+          session: sessionConfig,
           csrf: {
             cookie: {
               key: '_csrf',

website/app.test.js

@@ -58,6 +58,16 @@ describe('createAposConfig', () => {
     });
   });
 
+  test('uses in-memory session store when REDIS_URI is not set', () => {
+    delete process.env.REDIS_URI;
+
+    const config = createAposConfig();
+
+    expect(
+      config.modules['@apostrophecms/express'].options.session.store,
+    ).toBeUndefined();
+  });
+
   // Define module categories for verification - moved outside the test
   const moduleCategories = [
     {

website/utils/env.js

@@ -7,6 +7,15 @@ const getEnv = (name) => {
   return value;
 };
 
+const getOptionalEnv = (name, defaultValue) => {
+  const value = process.env[name];
+  if (value === undefined) {
+    return defaultValue;
+  }
+  return value;
+};
+
 module.exports = {
   getEnv,
+  getOptionalEnv,
 };

website/utils/env.test.js

@@ -1,4 +1,4 @@
-const { getEnv } = require('./env');
+const { getEnv, getOptionalEnv } = require('./env');
 
 describe('getEnv utility', () => {
   const OLD_ENV = process.env;
@@ -30,3 +30,30 @@ describe('getEnv utility', () => {
     }).toThrow('Environment variable "NON_EXISTENT_VAR" is not defined');
   });
 });
+
+describe('getOptionalEnv utility', () => {
+  const OLD_ENV = process.env;
+
+  beforeEach(() => {
+    jest.resetModules();
+    process.env = { ...OLD_ENV };
+  });
+
+  afterAll(() => {
+    process.env = OLD_ENV;
+  });
+
+  test('should return environment variable value when it exists', () => {
+    process.env.OPTIONAL_TEST_VAR = 'optional-test-value';
+
+    const result = getOptionalEnv('OPTIONAL_TEST_VAR');
+
+    expect(result).toBe('optional-test-value');
+  });
+
+  test('should return default value when environment variable does not exist', () => {
+    const result = getOptionalEnv('NON_EXISTENT_OPTIONAL_VAR', 'default');
+
+    expect(result).toBe('default');
+  });
+});