Added support for HEAD requests, closes socketio#557

lib/static.js

@@ -272,7 +272,13 @@ Static.prototype.write = function (path, req, res) {
   function write (status, headers, content, encoding) {
     try {
       res.writeHead(status, headers || undefined);
-      res.end(content || '', encoding || undefined);
+
+      // only write content if it's not a HEAD request and we actually have
+      // some content to write (304's doesn't have content).
+      res.end(
+          req.method !== 'HEAD' && content ? content : ''
+        , encoding || undefined
+      );
     } catch (e) {}
   }
 
@@ -306,10 +312,10 @@ Static.prototype.write = function (path, req, res) {
     if (versioned) {
       var expires = self.manager.get('browser client expires')
       headers['Cache-Control'] = 'public, max-age=' + expires;
+      headers['Date'] = new Date().toUTCString();
       headers['Expires'] = new Date(Date.now() + (expires * 1000)).toUTCString();
     }
 
-    // check if we can send gzip data
     if (gzip && reply.gzip) {
       headers['Content-Length'] = reply.gzip.length;
       headers['Content-Encoding'] = 'gzip';

test/common.js

@@ -147,6 +147,24 @@ HTTPClient.prototype.post = function (path, data, opts, fn) {
   return this.request(path, opts, fn);
 };
 
+/**
+ * Issue a HEAD request
+ *
+ * @api private
+ */
+
+HTTPClient.prototype.head = function (path, opts, fn) {
+  if ('function' == typeof opts) {
+    fn = opts;
+    opts = {};
+  }
+
+  opts = opts || {};
+  opts.method = 'HEAD';
+
+  return this.request(path, opts, fn);
+};
+
 /**
  * Performs a handshake (GET) request
  *

test/static.test.js

@@ -455,6 +455,22 @@ module.exports = {
       io.server.close();
       done();
     });
-  }
+  },
+
+  'test that HEAD requests work': function (done) {
+    var port = ++ports
+      , io = sio.listen(port)
+      , cl = client(port);
+
+    cl.head('/socket.io/socket.io.js', function (res, data) {
+      res.headers['content-type'].should.eql('application/javascript');
+      res.headers['content-length'].should.match(/([0-9]+)/);
 
+      data.should.eql('');
+
+      cl.end();
+      io.server.close()
+      done();
+    });
+  }
 };