nodejs
diff --git a/Collapse file
‎doc/api/https.md‎
Copy file name to clipboardExpand all lines: doc/api/https.md
+10-2Lines changed: 10 additions & 2 deletions
Display the source diff
Display the rich diff b/Collapse file
‎doc/api/https.md‎
Copy file name to clipboardExpand all lines: doc/api/https.md
+10-2Lines changed: 10 additions & 2 deletions
Display the source diff
Display the rich diff
diff --git a/Collapse file
‎lib/_http_agent.js‎
Copy file name to clipboardExpand all lines: lib/_http_agent.js
+3Lines changed: 3 additions & 0 deletions b/Collapse file
‎lib/_http_agent.js‎
Copy file name to clipboardExpand all lines: lib/_http_agent.js
+3Lines changed: 3 additions & 0 deletions
diff --git a/Collapse file
‎test/parallel/test-https-simple.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-https-simple.js
+3Lines changed: 3 additions & 0 deletions b/Collapse file
‎test/parallel/test-https-simple.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-https-simple.js
+3Lines changed: 3 additions & 0 deletions
@@ -24,15 +24,23 @@ An [`Agent`][] object for HTTPS similar to [`http.Agent`][]. See
 [`https.request()`][] for more information.
 
 ### new Agent([options])
-
+<!-- YAML
+changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/28209
+    description: do not automatically set servername if the target host was
+                 specified using an IP address.
+-->
 * `options` {Object} Set of configurable options to set on the agent.
   Can have the same fields as for [`http.Agent(options)`][], and
   * `maxCachedSessions` {number} maximum number of TLS cached sessions.
     Use `0` to disable TLS session caching. **Default:** `100`.
   * `servername` {string} the value of
     [Server Name Indication extension][sni wiki] to be sent to the server. Use
     empty string `''` to disable sending the extension.
-    **Default:** hostname or IP address of the target server.
+    **Default:** hostname of the target server, unless the target server
+    is specified using an IP address, in which case the default is `''` (no
+    extension).
 
     See [`Session Resumption`][] for infomation about TLS session reuse.
 
 
@@ -256,6 +256,9 @@ function calculateServerName(options, req) {
       servername = hostHeader.split(':', 1)[0];
     }
   }
+  // Don't implicitly set invalid (IP) servernames.
+  if (net.isIP(servername))
+    servername = '';
   return servername;
 }
 
 
@@ -29,6 +29,9 @@ if (!common.hasCrypto)
 const assert = require('assert');
 const https = require('https');
 
+// Assert that the IP-as-servername deprecation warning does not occur.
+process.on('warning', common.mustNotCall());
+
 const options = {
   key: fixtures.readKey('agent1-key.pem'),
   cert: fixtures.readKey('agent1-cert.pem')
Original file line number	Diff line number	Diff line change
`@@ -256,6 +256,9 @@ function calculateServerName(options, req) {`
`256`	`256`	`servername = hostHeader.split(':', 1)[0];`
`257`	`257`	`}`
`258`	`258`	`}`
	`259`	`+ // Don't implicitly set invalid (IP) servernames.`
	`260`	`+ if (net.isIP(servername))`
	`261`	`+ servername = '';`
`259`	`262`	`return servername;`
`260`	`263`	`}`
`261`	`264`