Merge branch 'envman-master'

Author: pocesar

README.md

@@ -327,6 +327,8 @@ The main reason for this part of the module is that you can setup a stratum prox
 
 You may also test your pool sending arbitrary test data to see if it's responding properly.
 
+If your URL starts with 'stratum+tcp://', remove it!
+
 ```js
 var client = stratum.Client.create();
 

examples/client.js

@@ -3,9 +3,9 @@ var stratum = require('../lib');
 var client = stratum.Client.$create();
 
  //must be specified per EventEmitter requirements
-client.on('error', function(socket){
+client.on('error', function(socket, err){
   socket.destroy();
-  console.log('Connection closed');
+  console.log('Connection closed with error: ', err);
   process.exit(1);
 });
 
@@ -79,4 +79,4 @@ client.connect({
   })
   // this can be chained and is the last callback to execute
   //.done(function(){ });
-;
+;

lib/client.js

@@ -39,9 +39,9 @@ module.exports = function (classes){
         }
       });
 
-      self.socket.on('error', function clientSocketError(){
+      self.socket.on('error', function clientSocketError(err){
         if (self.emit) {
-          self.emit('error', self);
+          self.emit('error', self, err);
         }
       });
 
@@ -324,4 +324,4 @@ module.exports = function (classes){
   });
 
   return Client;
-};
+};

package.json

@@ -50,10 +50,10 @@
     "better-curry": "*"
   },
   "devDependencies": {
-    "mocha": "*",
+    "mocha": "3.5.3",
     "jshint": "*",
     "expect.js": "*",
-    "sinon": "*",
+    "sinon": "1.17.7",
     "istanbul": "*",
     "coveralls": "*",
     "mocha-lcov-reporter": "*"

test/tests.js

@@ -1243,4 +1243,4 @@ module.exports = {
       }
     }
   }
-};
+};