Docs imply that http globalAgent is created with keepAlive: false

[janeklb]

Affected URL(s)

https://nodejs.org/api/http.html#new-agentoptions

Description of the problem

The docs state:

The default http.globalAgent that is used by http.request() has all of these values set to their respective defaults.

Whereas globalAgent is built with

module.exports = {
  Agent,
  globalAgent: new Agent({ keepAlive: true, scheduling: 'lifo', timeout: 5000 }),
};

https://github.com/nodejs/node/blob/339eb10619ada1b482371ee244585de7f4a58552/lib/_http_agent.js#L552C1-L555C3

the only default there is scheduling: 'lifo'; keepAlive: true and timeout: 5000 are overrides.

---

(the history part of the globalAgent docs states "The agent now uses HTTP Keep-Alive by default." so that part is ok ✔️)

[heysujal]

@VoltrexKeyva here is my understanding using scheduling : 'lifo' is not an override but it's not actually required because it is setting the same value again.
Also, I think we should add info only about the override for keepAlive and timeout here https://nodejs.org/api/http.html#httpglobalagent
So, we can mention something like
"Global instance of Agent which is used as the default for all HTTP client requests with the following overrides"
keepAlive : true
timeout : 5000

I would like to create a PR for this upon confirmation on above info.