feat: avoid cache on added projects

Author: rofe

src/extension/auth.js

@@ -19,9 +19,6 @@ import { ADMIN_ORIGIN, ADMIN_ORIGIN_NEW } from './utils/admin.js';
 const { host: adminHost } = new URL(ADMIN_ORIGIN);
 const { host: newAdminHost } = new URL(ADMIN_ORIGIN_NEW);
 
-/** Cache-Control max-age in seconds for added project hosts (HTML, JSON and code): 1 minute. */
-export const CACHE_MAX_AGE_SECONDS = 60;
-
 function getRandomId() {
   return Math.floor(Math.random() * 1000000);
 }
@@ -59,10 +56,14 @@ export function getCacheControlRules(projectConfigs) {
         priority: 1,
         action: {
           type: 'modifyHeaders',
-          responseHeaders: [{
+          requestHeaders: [{
+            operation: 'set',
             header: 'Cache-Control',
+            value: 'no-cache',
+          }, {
             operation: 'set',
-            value: `max-age=${CACHE_MAX_AGE_SECONDS}, must-revalidate`,
+            header: 'Pragma',
+            value: 'no-cache',
           }],
         },
         condition: {

test/auth.test.js

@@ -18,7 +18,6 @@ import { setUserAgent } from '@web/test-runner-commands';
 import sinon from 'sinon';
 
 import {
-  CACHE_MAX_AGE_SECONDS,
   configureAuthAndCorsHeaders,
   getCacheControlRules,
   getHostDomain,
@@ -82,8 +81,8 @@ describe('Test auth', () => {
       ];
       const rules = getCacheControlRules(configs);
       expect(rules).to.have.lengthOf(2);
-      expect(rules.every((r) => r.action?.responseHeaders?.[0]?.header === 'Cache-Control')).to.be.true;
-      expect(rules.every((r) => r.action.responseHeaders[0].value === `max-age=${CACHE_MAX_AGE_SECONDS}, must-revalidate`)).to.be.true;
+      expect(rules.every((r) => r.action?.requestHeaders?.some((h) => h.header === 'Cache-Control' && h.value === 'no-cache'))).to.be.true;
+      expect(rules.every((r) => r.action?.requestHeaders?.some((h) => h.header === 'Pragma' && h.value === 'no-cache'))).to.be.true;
       const filters = rules.map((r) => r.condition.regexFilter);
       expect(filters).to.include('^https://prod\\.example\\.com/.*');
       expect(filters).to.include('^https://live\\.example\\.com/.*');