From 2b8c2f649209f959c363279a1064fe0366968858 Mon Sep 17 00:00:00 2001
From: Matteo Collina <hello@matteocollina.com>
Date: Wed, 30 Dec 2020 13:02:27 +0100
Subject: [PATCH] http: set lifo as the default scheduling strategy in Agent.

---
 doc/api/http.md                             | 5 ++++-
 lib/_http_agent.js                          | 2 +-
 test/parallel/test-http-agent-scheduling.js | 6 +++---
 3 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/doc/api/http.md b/doc/api/http.md
index ad34b6e1e7df40..25107674074fd8 100644
--- a/doc/api/http.md
+++ b/doc/api/http.md
@@ -113,6 +113,9 @@ http.get({
 <!-- YAML
 added: v0.3.4
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/36685
+    description: Change the default scheduling from 'fifo' to 'lifo'.
   - version:
     - v14.5.0
     - v12.19.0
@@ -161,7 +164,7 @@ changes:
     In case of a high rate of request per second,
     the `'fifo'` scheduling will maximize the number of open sockets,
     while the `'lifo'` scheduling will keep it as low as possible.
-    **Default:** `'fifo'`.
+    **Default:** `'lifo'`.
   * `timeout` {number} Socket timeout in milliseconds.
     This will set the timeout when the socket is created.
 
diff --git a/lib/_http_agent.js b/lib/_http_agent.js
index 10638fe69ed124..1875d2ca821644 100644
--- a/lib/_http_agent.js
+++ b/lib/_http_agent.js
@@ -95,7 +95,7 @@ function Agent(options) {
   this.keepAlive = this.options.keepAlive || false;
   this.maxSockets = this.options.maxSockets || Agent.defaultMaxSockets;
   this.maxFreeSockets = this.options.maxFreeSockets || 256;
-  this.scheduling = this.options.scheduling || 'fifo';
+  this.scheduling = this.options.scheduling || 'lifo';
   this.maxTotalSockets = this.options.maxTotalSockets;
   this.totalSocketCount = 0;
 
diff --git a/test/parallel/test-http-agent-scheduling.js b/test/parallel/test-http-agent-scheduling.js
index b5ed2df4363a55..768519d5977831 100644
--- a/test/parallel/test-http-agent-scheduling.js
+++ b/test/parallel/test-http-agent-scheduling.js
@@ -56,11 +56,11 @@ function defaultTest() {
 
     bulkRequest(url, agent, (ports) => {
       makeRequest(url, agent, (port) => {
-        assert.strictEqual(ports[0], port);
+        assert.strictEqual(ports[ports.length - 1], port);
         makeRequest(url, agent, (port) => {
-          assert.strictEqual(ports[1], port);
+          assert.strictEqual(ports[ports.length - 1], port);
           makeRequest(url, agent, (port) => {
-            assert.strictEqual(ports[2], port);
+            assert.strictEqual(ports[ports.length - 1], port);
             server.close();
             agent.destroy();
           });