Cache logged out account-nav.json and session

Cache ^/fragment/account-nav.json and ^/session for logged out users
scratchfoundation · Dec 1, 2017 · 0ba2693 · 0ba2693
1 parent fd6211b
commit 0ba2693
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/bin/configure-fastly.js b/bin/configure-fastly.js
@@ -67,7 +67,8 @@ async.auto({
                          'std.tolower(req.http.Accept-Language)' +
                      ');\n' +
             '    }\n' +
-            '    if (req.url ~ "^/projects/" && !req.http.Cookie:scratchsessionsid) {\n' +
+            '    if (req.url ~ "^(/projects/|/fragment/account-nav.json|/session/)" && ' +
+            '!req.http.Cookie:scratchsessionsid) {\n' +
             '        set req.http.Cookie = "scratchlanguage=" req.http.Cookie:scratchlanguage;\n' +
             '    } else {\n' +
             '        return(pass);\n' +

diff --git a/bin/lib/fastly-config-methods.js b/bin/lib/fastly-config-methods.js
@@ -93,7 +93,8 @@ var FastlyConfigMethods = {
     setResponseTTL: function (condition) {
         return '' +
             'if (' + condition + ') {\n' +
-            '    if (req.url ~ "^/projects/" && !req.http.Cookie:scratchsessionsid) {\n' +
+            '    if (req.url ~ "^(/projects/|/fragment/account-nav.json|/session/)" && ' +
+            '!req.http.Cookie:scratchsessionsid) {\n' +
             '        set beresp.http.Vary = "Accept-Encoding, Accept-Language";\n' +
             '        unset beresp.http.set-cookie;\n' +
             '        return(deliver);\n' +

diff --git a/test/unit/test_fastly_config_methods.js b/test/unit/test_fastly_config_methods.js
@@ -63,7 +63,8 @@ tap.test('testSetTTL', function (t) {
     var ttl = fastlyConfig.setResponseTTL('itsactuallyttyl');
     t.equal(ttl, '' +
         'if (itsactuallyttyl) {\n' +
-        '    if (req.url ~ "^/projects/" && !req.http.Cookie:scratchsessionsid) {\n' +
+        '    if (req.url ~ "^(/projects/|/fragment/account-nav.json|/session/)" && ' +
+        '!req.http.Cookie:scratchsessionsid) {\n' +
         '        set beresp.http.Vary = "Accept-Encoding, Accept-Language";\n' +
         '        unset beresp.http.set-cookie;\n' +
         '        return(deliver);\n' +