tests fix

Author: karpikpl

.github/workflows/playwright.yml

@@ -56,7 +56,7 @@ jobs:
           -e SESSION_SECRET=dummy \
           api:test "@org"
     - uses: actions/upload-artifact@v4
-      if: ${{ !cancelled() }}
+      if: always()
       with:
         name: playwright-report-docker-org
         path: test-results-docker-org/
@@ -89,7 +89,7 @@ jobs:
           -e SESSION_SECRET=dummy \
           api:test "@ent"
     - uses: actions/upload-artifact@v4
-      if: ${{ !cancelled() }}
+      if: always()
       with:
         name: playwright-report-docker-ent
         path: test-results-docker-ent/

api/server.mjs

@@ -17,6 +17,10 @@ const MemoryStore = MemoryStoreFactory(session);
 const limiter = RateLimit({
   windowMs: 15 * 60 * 1000, // 15 minutes
   max: 100, // max 100 requests per windowMs
+  skip: (req) => {
+    // Skip rate limiting for localhost
+    return isLocalhost(req);
+  }
 });
 
 if (DOTENV_CONFIG_PATH) {
@@ -27,8 +31,13 @@ if (DOTENV_CONFIG_PATH) {
 }
 
 const app = express();
+console.log('ENVIRONMENT: ', app.get('env'));
+
+// Disable rate limiter and secure cookies for localhost
+const isLocalhost = (req) => {
+  return req.hostname === 'localhost' || req.hostname === '127.0.0.1';
+};
 
-// apply rate limiter to all requests
 app.use(limiter);
 
 app.use(session({
@@ -38,7 +47,8 @@ app.use(session({
   store: new MemoryStore({
     checkPeriod: 86400000 // prune expired entries every 24h
   }),
-  cookie: { secure: process.env.IS_PROD ? true : false, maxAge: 86400000 }
+  // may need to use secure: false if using http during local development
+  cookie: { secure: app.get('env') === 'production', maxAge: 86400000 }
 }));
 
 // Middleware to add Authorization header

playwright.docker.config.ts

@@ -3,6 +3,7 @@ import baseConfig from './playwright.config';
 
 export default defineConfig({
   ...baseConfig,
+  timeout: 5 * 1000,
   reporter: [
     ['list'],
     ['html', { open: 'never', outputFolder: '/test-results/html' }],