Cleanup the testing and old code related to Fastly headers (#17643)

Author: JamesMGreene

middleware/set-fastly-cache-headers.js

@@ -1,45 +1,8 @@
-const FASTLY_TTL = process.env.FASTLY_TTL || String(60 * 60 * 24) // 24 hours
-const STALE_TTL = String(60 * 10) // 10 minutes
-
 module.exports = (req, res, next) => {
-  const BYPASS_FASTLY = process.env.TEST_BYPASS_FASTLY === 'true'
-
-  // Bypass Fastly caching for all rendered pages
-  if (BYPASS_FASTLY) {
-    // Disallow both Fastly AND the browser from caching HTML pages
-    res.set({
-      'surrogate-control': 'private, no-store',
-      'cache-control': 'private, no-store'
-    })
-    return next()
-  }
-
+  // Disallow both Fastly AND the browser from caching HTML pages
   res.set({
-
-    // Say you want Fastly to cache your content but you don't want it cached by browsers.
-    // The best way to do this would be to send Fastly both the Cache-Control header as you want
-    // it to go to the browsers, and use Surrogate-Control to tell us how long to cache for.
-
-    // Fastly does not currently respect no-store or no-cache directives.
-    // Including either or both of these in a Cache-Control header has no effect on
-    // Fastly's caching decision, unless you alter this behavior using custom VCL.
-
-    // https://docs.fastly.com/en/guides/configuring-caching
-    'cache-control': 'no-store, must-revalidate',
-
-    // This header gets stripped and is only visible to Fastly caches.
-    // https://docs.fastly.com/en/guides/serving-stale-content#manually-enabling-serve-stale
-    'surrogate-control': `max-age=${FASTLY_TTL}, stale-if-error=${STALE_TTL}, stale-while-revalidate=${STALE_TTL}`,
-
-    // Fastly provides a Soft Purge feature that allows you to mark content as outdated (stale) instead of permanently
-    // purging and thereby deleting it from Fastly's caches. Objects invalidated with Soft Purge will be treated as
-    // outdated (stale) while Fastly fetches a new version from origin.
-    //
-    // Use of a surrogate key is required for soft purging
-    // https://docs.fastly.com/en/guides/soft-purges
-    // https://docs.fastly.com/en/guides/getting-started-with-surrogate-keys
-    'surrogate-key': 'all-the-things'
+    'surrogate-control': 'private, no-store',
+    'cache-control': 'private, no-store'
   })
-
-  next()
+  return next()
 }

tests/rendering/server.js

@@ -59,16 +59,7 @@ describe('server', () => {
     expect(csp.get('style-src').includes("'unsafe-inline'")).toBe(true)
   })
 
-  test('sets Fastly cache control headers', async () => {
-    const res = await get('/en')
-    expect(res.headers['cache-control']).toBe('no-store, must-revalidate')
-    expect(res.headers['surrogate-control']).toBe('max-age=86400, stale-if-error=600, stale-while-revalidate=600')
-    expect(res.headers['surrogate-key']).toBe('all-the-things')
-  })
-
-  test('sets Fastly cache control headers to bypass if enabled', async () => {
-    process.env.TEST_BYPASS_FASTLY = 'true'
-
+  test('sets Fastly cache control headers to bypass pages', async () => {
     const res = await get('/en')
     expect(res.headers['cache-control']).toBe('private, no-store')
     expect(res.headers['surrogate-control']).toBe('private, no-store')