nock 13.0.6 → 13.0.10

package/README.md

@@ -1213,13 +1213,12 @@ nocks.forEach(function (nock) {
     const recordedBodyResult = /timestamp:([0-9]+)/.exec(aRecordedBody)
     if (recordedBodyResult) {
       const recordedTimestamp = recordedBodyResult[1]
-      return body.replace(/(timestamp):([0-9]+)/g, function (
-        match,
-        key,
-        value
-      ) {
-        return key + ':' + recordedTimestamp
-      })
+      return body.replace(
+        /(timestamp):([0-9]+)/g,
+        function (match, key, value) {
+          return key + ':' + recordedTimestamp
+        }
+      )
     } else {
       return body
     }

package/lib/common.js

@@ -673,6 +673,34 @@ function removeAllTimers() {
   clearTimer(clearImmediate, immediates)
 }
 
+/**
+ * Check if the Client Request has been cancelled.
+ *
+ * Until Node 14 is the minimum, we need to look at both flags to see if the request has been cancelled.
+ * The two flags have the same purpose, but the Node maintainers are migrating from `abort(ed)` to
+ * `destroy(ed)` terminology, to be more consistent with `stream.Writable`.
+ * In Node 14.x+, Calling `abort()` will set both `aborted` and `destroyed` to true, however,
+ * calling `destroy()` will only set `destroyed` to true.
+ * Falling back on checking if the socket is destroyed to cover the case of Node <14.x where
+ * `destroy()` is called, but `destroyed` is undefined.
+ *
+ * Node Client Request history:
+ * - `request.abort()`: Added in: v0.3.8, Deprecated since: v14.1.0, v13.14.0
+ * - `request.aborted`: Added in: v0.11.14, Became a boolean instead of a timestamp: v11.0.0, Not deprecated (yet)
+ * - `request.destroy()`: Added in: v0.3.0
+ * - `request.destroyed`: Added in: v14.1.0, v13.14.0
+ *
+ * @param {ClientRequest} req
+ * @returns {boolean}
+ */
+function isRequestDestroyed(req) {
+  return !!(
+    req.destroyed === true ||
+    req.aborted ||
+    (req.socket && req.socket.destroyed)
+  )
+}
+
 module.exports = {
   contentEncoding,
   dataEqual,
@@ -686,6 +714,7 @@ module.exports = {
   isContentEncoded,
   isJSONContent,
   isPlainObject,
+  isRequestDestroyed,
   isStream,
   isUtf8Representable,
   mapValue,

package/lib/intercepted_request_router.js

@@ -103,8 +103,7 @@ class InterceptedRequestRouter {
   connectSocket() {
     const { req, socket } = this
 
-    // Until Node 14 is the minimum, we need to look at both flags to see if the request has been cancelled.
-    if (req.destroyed || req.aborted) {
+    if (common.isRequestDestroyed(req)) {
       return
     }
 
@@ -250,8 +249,7 @@ class InterceptedRequestRouter {
       return
     }
 
-    // Until Node 14 is the minimum, we need to look at both flags to see if the request has been cancelled.
-    if (!req.destroyed && !req.aborted && !playbackStarted) {
+    if (!common.isRequestDestroyed(req) && !playbackStarted) {
       this.startPlayback()
     }
   }

package/lib/playback_interceptor.js

@@ -293,7 +293,7 @@ function playbackInterceptor({
     const { delayBodyInMs, delayConnectionInMs } = interceptor
 
     function respond() {
-      if (req.aborted) {
+      if (common.isRequestDestroyed(req)) {
         return
       }
 
@@ -318,7 +318,7 @@ function playbackInterceptor({
   // correct events are emitted first ('socket', 'finish') and any aborts in the in the queue or
   // called during a 'finish' listener can be called.
   common.setImmediate(() => {
-    if (!req.aborted) {
+    if (!common.isRequestDestroyed(req)) {
       start()
     }
   })

package/lib/recorder.js

@@ -110,6 +110,10 @@ function generateRequestAndResponse({
     const queryStr = req.path.slice(queryIndex + 1)
     queryObj = querystring.parse(queryStr)
   }
+
+  // Escape any single quotes in the path as the output uses them
+  path = path.replace(/'/g, `\\'`)
+
   // Always encode the query parameters when recording.
   const encodedQueryObj = {}
   for (const key in queryObj) {
@@ -305,8 +309,6 @@ function record(recOptions) {
 
       if (callback) {
         callback(res, options, callback)
-      } else {
-        res.resume()
       }
 
       debug('finished setting up intercepting')

package/lib/socket.js

@@ -7,9 +7,12 @@ module.exports = class Socket extends EventEmitter {
   constructor(options) {
     super()
 
+    // Pretend this is a TLSSocket
     if (options.proto === 'https') {
       // https://github.com/nock/nock/issues/158
       this.authorized = true
+      // https://github.com/nock/nock/issues/2147
+      this.encrypted = true
     }
 
     this.bufferSize = 0

package/package.json

@@ -7,7 +7,7 @@
     "testing",
     "isolation"
   ],
-  "version": "13.0.6",
+  "version": "13.0.10",
   "author": "Pedro Teixeira <pedro.teixeira@gmail.com>",
   "repository": {
     "type": "git",
@@ -40,13 +40,13 @@
     "eslint-plugin-mocha": "^8.0.0",
     "eslint-plugin-node": "^11.0.0",
     "eslint-plugin-promise": "^4.1.1",
-    "eslint-plugin-standard": "^4.0.0",
-    "form-data": "^3.0.0",
+    "eslint-plugin-standard": "^5.0.0",
+    "form-data": "^4.0.0",
     "got": "^11.3.0",
     "mocha": "^8.0.1",
     "npm-run-all": "^4.1.5",
     "nyc": "^15.0.0",
-    "prettier": "2.0.5",
+    "prettier": "2.2.1",
     "proxyquire": "^2.1.0",
     "request": "^2.83.0",
     "rimraf": "^3.0.0",
@@ -54,7 +54,7 @@
     "sinon": "^9.0.0",
     "sinon-chai": "^3.3.0",
     "superagent": "^6.1.0",
-    "tap": "14.6.1"
+    "typescript": "^4.2.2"
   },
   "scripts": {
     "format:fix": "prettier --write '**/*.{js,json,md,ts,yml,yaml}'",
@@ -63,19 +63,8 @@
     "lint:js": "eslint --cache --cache-location './.cache/eslint' '**/*.js'",
     "lint:js:fix": "eslint --cache --cache-location './.cache/eslint' --fix '**/*.js'",
     "lint:ts": "dtslint types",
-    "test": "run-s test:mocha test:tap",
-    "test:coverage": "tap --coverage-report=html && open coverage/lcov-report/index.html",
-    "test:mocha": "nyc mocha $(grep -lr '^\\s*it(' tests)",
-    "test:tap": "tap --100 --coverage --coverage-report=text ./tests/test_*.js"
-  },
-  "nyc": {
-    "reporter": [
-      "lcov",
-      "text-summary"
-    ],
-    "exclude": [
-      "tests/"
-    ]
+    "test": "nyc mocha tests",
+    "test:coverage": "open coverage/lcov-report/index.html"
   },
   "license": "MIT",
   "files": [