feat: capture error stack traces (nodejs#1619)

Author: sidwebworks

index-fetch.js

@@ -5,7 +5,12 @@ const fetchImpl = require('./lib/fetch')
 
 module.exports.fetch = async function fetch (resource) {
   const dispatcher = (arguments[1] && arguments[1].dispatcher) || getGlobalDispatcher()
-  return fetchImpl.apply(dispatcher, arguments)
+  try {
+    return await fetchImpl.apply(dispatcher, arguments)
+  } catch (err) {
+    Error.captureStackTrace(err, this)
+    throw err
+  }
 }
 module.exports.FormData = require('./lib/fetch/formdata').FormData
 module.exports.Headers = require('./lib/fetch/headers').Headers

index.js

@@ -92,7 +92,12 @@ if (nodeMajor > 16 || (nodeMajor === 16 && nodeMinor >= 8)) {
       fetchImpl = require('./lib/fetch')
     }
     const dispatcher = (arguments[1] && arguments[1].dispatcher) || getGlobalDispatcher()
-    return fetchImpl.apply(dispatcher, arguments)
+    try {
+      return await fetchImpl.apply(dispatcher, arguments)
+    } catch (err) {
+      Error.captureStackTrace(err, this)
+      throw err
+    }
   }
   module.exports.Headers = require('./lib/fetch/headers').Headers
   module.exports.Response = require('./lib/fetch/response').Response

test/fetch/client-error-stack-trace.js

@@ -0,0 +1,21 @@
+'use strict'
+
+const { test } = require('tap')
+const { fetch } = require('../..')
+const { fetch: fetchIndex } = require('../../index-fetch')
+
+test('FETCH: request errors and prints trimmed stack trace', async (t) => {
+  try {
+    await fetch('http://a.com')
+  } catch (error) {
+    t.match(error.stack, `at Test.<anonymous> (${__filename}`)
+  }
+})
+
+test('FETCH-index: request errors and prints trimmed stack trace', async (t) => {
+  try {
+    await fetchIndex('http://a.com')
+  } catch (error) {
+    t.match(error.stack, `at Test.<anonymous> (${__filename}`)
+  }
+})