nock 9.0.8 → 9.0.9

package/CHANGELOG.md

@@ -1,3 +1,26 @@
+9.0.8 / 2017-02-27
+==================
+
+  * 9.0.8: Fix for [#754](https://github.com/node-nock/nock/issues/754)
+  * Merge pull request [#836](https://github.com/node-nock/nock/issues/836) from node-nock/greenkeeper-superagent-3.5.0
+    Update superagent to version 3.5.0 🚀
+  * Merge pull request [#802](https://github.com/node-nock/nock/issues/802) from aleung/fix-754
+    Fix request timeout no working
+
+9.0.7 / 2017-02-23
+==================
+
+  * Changelog
+  * 9.0.7: Fix for [#610](https://github.com/node-nock/nock/issues/610)
+  * Merge pull request [#829](https://github.com/node-nock/nock/issues/829) from yodeyer/fixQs
+    Fix interceptor if query have no query strings
+  * chore(package): update superagent to version 3.5.0
+    https://greenkeeper.io/
+  * Test against request 2.79.0, with which timeout issue occured.
+  * Add Visual Studio Code folder into gitignore
+  * Update the failed test case
+  * Fix https://github.com/node-nock/nock/issues/754
+
 9.0.6 / 2017-02-13
 ==================
 
@@ -153,19 +176,3 @@
   * Make pendingMocks and isDone behave consistently
   * Removed coverage folder from npm
   * Update PendingMocks when an interceptor is removed
-  * Update CHANGELOG
-
-8.1.0 / 2016-10-10
-==================
-
-  * 8.1.0
-  * Enable harmony mode when running tests
-  * Remove duplicated Interceptor.replyWithFile function
-  * Merge pull request [#710](https://github.com/node-nock/nock/issues/710) from hugoduraes/master
-    Leave rawHeaders case untouched on nock's replies
-  * Leave rawHeaders case untouched on nock's replies
-  * Re-enable previously skipped tests
-  * Merge pull request [#586](https://github.com/node-nock/nock/issues/586) from Elergy/master
-    Fix some broken tests
-  * Add nodejs v6 to travis config
-  * use public TravisCI badge icon, which shows failing tests.

package/lib/request_overrider.js

@@ -133,7 +133,6 @@ function RequestOverrider(req, options, interceptors, remove, cb) {
   };
 
   req.socket = response.socket = Socket({ proto: options.proto });
-  req.socket.connecting = true;
 
   req.write = function(buffer, encoding) {
     debug('write', arguments);
@@ -195,6 +194,8 @@ function RequestOverrider(req, options, interceptors, remove, cb) {
     // emit a fake socket.
     if (event == 'socket') {
       listener.call(req, req.socket);
+      req.socket.emit('connect', req.socket);
+      req.socket.emit('secureConnect', req.socket);
     }
 
     EventEmitter.prototype.on.call(this, event, listener);
@@ -490,9 +491,6 @@ function RequestOverrider(req, options, interceptors, remove, cb) {
 
           debug('emitting response');
 
-          req.socket.emit('connect', req.socket);
-          req.socket.emit('secureConnect', req.socket);
-
           if (typeof cb === 'function') {
             debug('callback with response');
             cb(response);

package/node_modules/changelog/package.json

@@ -28,7 +28,7 @@
   "_requiredBy": [
     "#DEV:/"
   ],
-  "_resolved": "https://npme.walmart.com/changelog/-/changelog-1.0.7.tgz",
+  "_resolved": "https://registry.npmjs.org/changelog/-/changelog-1.0.7.tgz",
   "_shasum": "e95de6dc14082948bd49618304e9c9e854e821e5",
   "_shrinkwrap": null,
   "_spec": "changelog@^1.0.7",

package/package.json

@@ -7,7 +7,7 @@
     "testing",
     "isolation"
   ],
-  "version": "9.0.8",
+  "version": "9.0.9",
   "author": "Pedro Teixeira <pedro.teixeira@gmail.com>",
   "contributors": [
     {
@@ -191,7 +191,7 @@
     "node-static": "^0.7.7",
     "nyc": "^10.0.0",
     "pre-commit": "1.1.2",
-    "request": "2.79.0",
+    "request": "2.71.0",
     "request-promise": "^2.0.1",
     "restify": "^4.0.4",
     "restler": "3.4.0",

package/tests/browserify-public/browserify-bundle.js

@@ -2355,7 +2355,6 @@ function RequestOverrider(req, options, interceptors, remove, cb) {
   };
 
   req.socket = response.socket = Socket({ proto: options.proto });
-  req.socket.connecting = true;
 
   req.write = function(buffer, encoding) {
     debug('write', arguments);
@@ -2417,6 +2416,8 @@ function RequestOverrider(req, options, interceptors, remove, cb) {
     // emit a fake socket.
     if (event == 'socket') {
       listener.call(req, req.socket);
+      req.socket.emit('connect', req.socket);
+      req.socket.emit('secureConnect', req.socket);
     }
 
     EventEmitter.prototype.on.call(this, event, listener);
@@ -2712,9 +2713,6 @@ function RequestOverrider(req, options, interceptors, remove, cb) {
 
           debug('emitting response');
 
-          req.socket.emit('connect', req.socket);
-          req.socket.emit('secureConnect', req.socket);
-
           if (typeof cb === 'function') {
             debug('callback with response');
             cb(response);

package/tests/test_intercept.js

@@ -4234,12 +4234,12 @@ test('socket emits connect and secureConnect', function(t) {
 
   req.on('socket', function(socket) {
     socket.once('connect', function() {
+      req.end();
       t.ok(true);
     });
     socket.once('secureConnect', function() {
       t.ok(true);
     });
-    req.end();
   });
 
   req.once('response', function(res) {