Append to existing Vary header if it exists

Author: Mikko Tiihonen

index.js

@@ -694,7 +694,11 @@ SendStream.prototype.sendFile = function sendFile(path) {
   }
 
   function sendPreferredContent(p, stat, contents) {
-    if (contents.length) self.res.setHeader('Vary', 'Accept-Encoding')
+    if (contents.length) {
+      var old = self.res.getHeader('Vary')
+      var value = (old ? old + ', ' : '') + 'Accept-Encoding'
+      self.res.setHeader('Vary', value)
+    }
 
     var preferredContent
     var extensions = self.getAcceptEncodingExtensions()

test/send.js

@@ -1174,6 +1174,16 @@ describe('send(file, options)', function(){
           .set('Accept-Encoding', 'gzip')
           .expect('Content-Length', '11', done)
     })
+
+    it('should append to existing Vary header', function(done){
+      var app = http.createServer(function(req, res){
+        res.setHeader('Vary', 'custom')
+        send(req, req.url, {precompressed: true, root: fixtures}).pipe(res)
+      });
+      request(app)
+         .get('/name.html')
+         .expect('Vary', 'custom, Accept-Encoding', done);
+    })
   })
 
   describe('index', function(){