socket: clear timer after sending one noop packet (fixes #174)

lib/socket.js

@@ -2,8 +2,8 @@
  * Module dependencies.
  */
 
-var EventEmitter = require('events').EventEmitter
-  , debug = require('debug')('engine:socket')
+var EventEmitter = require('events').EventEmitter;
+var debug = require('debug')('engine:socket');
 
 /**
  * Module exports.
@@ -177,14 +177,13 @@ Socket.prototype.maybeUpgrade = function (transport) {
       transport.send([{ type: 'pong', data: 'probe' }]);
 
       // we force a polling cycle to ensure a fast upgrade
-      function check () {
+      self.checkIntervalTimer = setInterval(function(){
         if ('polling' == self.transport.name && self.transport.writable) {
           debug('writing a noop packet to polling for fast upgrade');
           self.transport.send([{ type: 'noop' }]);
+          clearInterval(self.checkIntervalTimer);
         }
-      }
-
-      self.checkIntervalTimer = setInterval(check, 100);
+      }, 100);
     } else if ('upgrade' == packet.type && self.readyState == 'open') {
       debug('got upgrade packet - upgrading');
       self.upgraded = true;
@@ -327,7 +326,7 @@ Socket.prototype.flush = function () {
     var wbuf = this.writeBuffer;
     this.writeBuffer = [];
     if (!this.transport.supportsFraming) {
-      this.sentCallbackFn.push(this.packetsFn)
+      this.sentCallbackFn.push(this.packetsFn);
     } else {
       this.sentCallbackFn.push.apply(this.sentCallbackFn, this.packetsFn);
     }