Merge pull request #19 from HotelDon/development

timeout now supports value of "none", allowing one to disable the aut…

README.md

@@ -76,7 +76,7 @@ function postAuthenticate(socket, data) {
 }
 ```
 
-* `timeout`: The amount of millisenconds to wait for a client to authenticate before disconnecting it. Defaults to 1000.
+* `timeout`: The amount of millisenconds to wait for a client to authenticate before disconnecting it. Defaults to 1000. The value 'none' disables the timeout feature.
 
 ## Auth error messages
 

lib/socketio-auth.js

@@ -13,7 +13,7 @@ var debug = require('debug')('socketio-auth');
  * @param {Function} config.authenticate - indicates if authentication was successfull
  * @param {Function} config.postAuthenticate=noop -  called after the client is authenticated
  * @param {Number} [config.timeout=1000] - amount of millisenconds to wait for a client to
- * authenticate before disconnecting it
+ * authenticate before disconnecting it. A value of 'none' means no connection timeout.
  */
 module.exports = function socketIOAuth(io, config) {
   config = config || {};
@@ -53,13 +53,15 @@ module.exports = function socketIOAuth(io, config) {
 
     });
 
-    setTimeout(function() {
-      // If the socket didn't authenticate after connection, disconnect it
-      if (!socket.auth) {
-        debug('Disconnecting socket %s', socket.id);
-        socket.disconnect('unauthorized');
-      }
-    }, timeout);
+    if (timeout !== 'none') {
+      setTimeout(function() {
+          // If the socket didn't authenticate after connection, disconnect it
+          if (!socket.auth) {
+            debug('Disconnecting socket %s', socket.id);
+            socket.disconnect('unauthorized');
+          }
+        }, timeout);
+    }
 
   });
 };