fix(fetch): makeNetworkError called twice when httpNetworkFetch run error

[meixg]

fixes: nodejs/node#42804

the error will be warped again by makeNetworkError at L1751:

undici/lib/fetch/index.js

Lines 1734 to 1752 in b6a7262

	try {
	const { body, status, statusText, headersList } = await dispatch({ body: requestBody })
	const iterator = body[Symbol.asyncIterator]()
	fetchParams.controller.next = () => iterator.next()
	response = makeResponse({ status, statusText, headersList })
	} catch (err) {
	// 10. If aborted, then:
	if (err.name === 'AbortError') {
	// 1. If connection uses HTTP/2, then transmit an RST_STREAM frame.
	fetchParams.controller.connection.destroy()
	// 2. Return the appropriate network error for fetchParams.
	return makeAppropriateNetworkError(fetchParams)
	}
	return makeNetworkError(err)
	}

[codecov-commenter]

Codecov Report

Merging #1353 (88de74c) into main (b6a7262) will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1353      +/-   ##
==========================================
+ Coverage   94.12%   94.24%   +0.11%     
==========================================
  Files          45       45              
  Lines        4204     4204              
==========================================
+ Hits         3957     3962       +5     
+ Misses        247      242       -5     

Impacted Files	Coverage Δ
lib/fetch/index.js	80.85% <100.00%> (+0.55%)	⬆️
lib/fetch/response.js	88.00% <0.00%> (+1.33%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b6a7262...88de74c. Read the comment docs.

[ronag]

Nice catch!

[mcollina]

lgtm