Merge pull request #3 from Varkoff/04-authentication

Remove Redis Volume
Varkoff · Apr 20, 2024 · da79cd9 · da79cd9
2 parents abad676 + 362aea2
commit da79cd9
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 6 deletions.
diff --git a/backend/src/main.ts b/backend/src/main.ts
@@ -21,8 +21,9 @@ async function bootstrap() {
   const redisUrl = process.env.REDIS_URL || "redis://localhost:6379"
   const redisClient = new Redis(redisUrl, {
 
+  }).on('error', console.error).on('connect', () => {
+    console.log('Connected to Redis');
   });
-  redisClient.connect().catch(console.error)
 
   // Initialize store
   const redisStore = new RedisStore({
@@ -41,7 +42,7 @@ async function bootstrap() {
       cookie: {
         maxAge: 1000 * 60 * 60 * 24 * 30, // 30 days
         sameSite: "lax",
-        secure: false
+        secure: process.env.NODE_ENV === 'production',
       }
     }),
   )

diff --git a/cache/dump.rdb b/cache/dump.rdb
diff --git a/docker-compose.dev.yml b/docker-compose.dev.yml
@@ -2,7 +2,7 @@ services:
   monorepo_dev:
     environment:
       - REDIS_URL=redis://redis_dev:6379
-      - NODE_ENV
+      - NODE_ENV=development
       - DATABASE_URL
 
     container_name: nestjs-remix-monorepo-dev

diff --git a/docker-compose.prod.yml b/docker-compose.prod.yml
@@ -21,7 +21,7 @@ services:
     #   - '6379:6379'
     command: ["redis-server"] 
     # --save 20 1 --loglevel warning --requirepass eYVX7EwVmmxKPCDmwMtyKVge8oLd2t81
-    volumes: 
-      # - ./cache:/data
-      - /dev/volumes/nestjs-remix/production/sessions/:/data
+    # volumes: 
+    #   # - ./cache:/data
+    #   - /dev/volumes/nestjs-remix/production/sessions/:/data