T3C-559 Adds environment configuration to rate limits

Author: juggler434

deploy/cloudrun/express-server.yaml

@@ -87,6 +87,8 @@ spec:
                 secretKeyRef:
                   key: latest
                   name: analytics-api-key
+            - name: RATE_LIMIT_PREFIX
+              value: dev-PR_NUMBER
 
           resources:
             limits:

express-server/src/__tests__/analytics-integration.test.ts

@@ -59,6 +59,7 @@ describe("Analytics Integration with Common Package", () => {
     ANALYTICS_ENABLED: true,
     ANALYTICS_DEBUG: false,
     FIREBASE_ADMIN_PROJECT_ID: undefined,
+    RATE_LIMIT_PREFIX: "test",
     ...overrides,
   });
 

express-server/src/__tests__/featureFlagsIntegration.test.ts

@@ -53,6 +53,7 @@ const createMockEnv = (
   ANALYTICS_ENABLED: false,
   ANALYTICS_DEBUG: false,
   FIREBASE_ADMIN_PROJECT_ID: undefined,
+  RATE_LIMIT_PREFIX: "test",
 });
 
 // Mock validateEnv to return a valid test environment

express-server/src/routes/__tests__/create.csv-security.test.ts

@@ -90,6 +90,7 @@ describe("CSV Security in Create Route", () => {
           ANALYTICS_FLUSH_AT: 20,
           ANALYTICS_FLUSH_INTERVAL: 10000,
           ANALYTICS_DEBUG: false,
+          RATE_LIMIT_PREFIX: "test",
         },
       };
       // Mock logger for RequestWithLogger interface

express-server/src/server.ts

@@ -107,6 +107,8 @@ initializeAnalyticsClient(env);
 // This is added here so that the worker gets initialized. Queue is referenced in /create, so its initialized there.
 setupWorkers(connection, env.REDIS_QUEUE_NAME);
 
+const rateLimitPrefix = env.RATE_LIMIT_PREFIX;
+
 const defaultRateLimiter = rateLimit({
   windowMs: 15 * 60 * 1000, // 15 minutes
   max: 100, // Limit each IP to 100 requests per windowMs
@@ -119,7 +121,7 @@ const defaultRateLimiter = rateLimit({
   store: new RedisStore({
     sendCommand: (command: string, ...args: string[]) =>
       connection.call(command, ...args) as Promise<RedisReply>,
-    prefix: "rate-limit-default",
+    prefix: `${rateLimitPrefix}-rate-limit-default`,
   }),
 });
 
@@ -136,7 +138,7 @@ const reportRateLimiter = rateLimit({
   store: new RedisStore({
     sendCommand: (command: string, ...args: string[]) =>
       connection.call(command, ...args) as Promise<RedisReply>,
-    prefix: "rate-limit-report",
+    prefix: `${rateLimitPrefix}-rate-limit-report`,
   }),
 });
 

express-server/src/types/context.ts

@@ -135,6 +135,7 @@ export const env = z.object({
     .default("false")
     .transform((val) => val.toLowerCase() === "true"),
   FIREBASE_ADMIN_PROJECT_ID: z.string().optional(),
+  RATE_LIMIT_PREFIX: z.string().optional().default("dev"),
 });
 
 export type Env = z.infer<typeof env>;