fix: Remove the default connection close header. (#1736)

dhedey · web-flow · commit 8b3320d2a7c0 · 2023-07-25T13:49:44.000+02:00
Instead, we rely on the underlying http implementation in Node.js to handle this, as per the documentation at https://nodejs.org/api/http.html#new-agentoptions This fixes #1735 and likely replaces #1473 The original change introducing this provided no clear motivation for the override, and the implementation has since been changed to disable this header when an agent is provided, so I think there is sufficient evidence that removing this is the correct behaviour. af21ae6 7f68577
diff --git a/README.md b/README.md
@@ -539,7 +539,6 @@ If no values are set, the following request headers will be sent automatically:
 | ------------------- | ------------------------------------------------------ |
 | `Accept-Encoding`   | `gzip, deflate, br` (when `options.compress === true`) |
 | `Accept`            | `*/*`                                                  |
-| `Connection`        | `close` _(when no `options.agent` is present)_         |
 | `Content-Length`    | _(automatically calculated, if possible)_              |
 | `Host`              | _(host and port information from the target URI)_      |
 | `Transfer-Encoding` | `chunked` _(when `req.body` is a stream)_              |
@@ -558,6 +557,8 @@ The `agent` option allows you to specify networking related options which are ou
 
 See [`http.Agent`](https://nodejs.org/api/http.html#http_new_agent_options) for more information.
 
+If no agent is specified, the default agent provided by Node.js is used. Note that [this changed in Node.js 19](https://github.com/nodejs/node/blob/4267b92604ad78584244488e7f7508a690cb80d0/lib/_http_agent.js#L564) to have `keepalive` true by default. If you wish to enable `keepalive` in an earlier version of Node.js, you can override the agent as per the following code sample. 
+
 In addition, the `agent` option accepts a function that returns `http`(s)`.Agent` instance given current [URL](https://nodejs.org/api/url.html), this is useful during a redirection chain across HTTP and HTTPS protocol.
 
 ```js
diff --git a/src/request.js b/src/request.js
@@ -288,10 +288,6 @@ export const getNodeRequestOptions = request => {
 		agent = agent(parsedURL);
 	}
 
-	if (!headers.has('Connection') && !agent) {
-		headers.set('Connection', 'close');
-	}
-
 	// HTTP-network fetch step 4.2
 	// chunked encoding is handled by Node.js
 

Original file line number	Diff line number	Diff line change
`@@ -288,10 +288,6 @@ export const getNodeRequestOptions = request => {`
`288`	`288`	`agent = agent(parsedURL);`
`289`	`289`	`}`
`290`	`290`
`291`		`- if (!headers.has('Connection') && !agent) {`
`292`		`- headers.set('Connection', 'close');`
`293`		`- }`
`294`		`-`
`295`	`291`	`// HTTP-network fetch step 4.2`
`296`	`292`	`// chunked encoding is handled by Node.js`
`297`	`293`