Only remove headers when handling a cachify request. Doh. Fixes Issue#9

mozilla · Feb 17, 2012 · 9f6af01 · 9f6af01 · lloyd · Feb 17, 2012
1 parent b6957c4
commit 9f6af01
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/lib/connect-cachify.js b/lib/connect-cachify.js
@@ -87,14 +87,14 @@ exports.setup = function (assets, options) {
         resp.setHeader('Cache-Control', 'public, max-age=31536000');
         req.url = true_path;
       }
-    }
-    resp.on('header', function () {
-      // Relax other middleware... I got this one
-      ['ETag', 'Last-Modified'].forEach(function (header) {
-        if (resp.getHeader(header)) resp.removeHeader(header);
-        if (resp.getHeader(header)) resp.removeHeader(header);
+      resp.on('header', function () {
+        // Relax other middleware... I got this one
+        ['ETag', 'Last-Modified'].forEach(function (header) {
+          if (resp.getHeader(header)) resp.removeHeader(header);
+          if (resp.getHeader(header)) resp.removeHeader(header);
+        });
       });
-    });
+    }
     next();
   };
 };