Stomp set-cookie

set-cookie causes the cache to not cache the page. Since we’re not trying to track the user beyond, we don’t want to try resetting the cookie again with the same info
scratchfoundation · Dec 1, 2017 · c65b0ef · c65b0ef
1 parent 1083a7a
commit c65b0ef
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/bin/lib/fastly-config-methods.js b/bin/lib/fastly-config-methods.js
@@ -95,6 +95,8 @@ var FastlyConfigMethods = {
             'if (' + condition + ') {\n' +
             '    if (req.url ~ "^/projects/" && !req.http.Cookie:scratchsessionsid) {\n' +
             '        set beresp.http.Vary = "Accept-Encoding, Accept-Language";\n' +
+            '        unset beresp.http.set-cookie;\n' +
+            '        return(deliver);\n' +
             '    } else {\n' +
             '        set beresp.ttl = 0s;\n' +
             '        set beresp.grace = 0s;\n' +

diff --git a/test/unit/test_fastly_config_methods.js b/test/unit/test_fastly_config_methods.js
@@ -65,6 +65,8 @@ tap.test('testSetTTL', function (t) {
         'if (itsactuallyttyl) {\n' +
         '    if (req.url ~ "^/projects/" && !req.http.Cookie:scratchsessionsid) {\n' +
         '        set beresp.http.Vary = "Accept-Encoding, Accept-Language";\n' +
+        '        unset beresp.http.set-cookie;\n' +
+        '        return(deliver);\n' +
         '    } else {\n' +
         '        set beresp.ttl = 0s;\n' +
         '        set beresp.grace = 0s;\n' +