Merge pull request #1721 from LLK/hotfix/cache-nav-session-endpoints

[Master] Hotfix/cache nav session endpoints @rschamp said I should.

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' +

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' +

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' +