Set servername on tls connect - fixes porsager#543

porsager · porsager · commit 498f2aec9fa2 · 2023-03-06T07:08:48.000+01:00
diff --git a/deno/polyfills.js b/deno/polyfills.js
@@ -1,10 +1,12 @@
 /* global Deno */
 
 import { Buffer } from 'https://deno.land/std@0.132.0/node/buffer.ts'
+import { isIP } from 'https://deno.land/std@0.132.0/node/net.ts'
 
 const events = () => ({ data: [], error: [], drain: [], connect: [], secureConnect: [], close: [] })
 
 export const net = {
+  isIP,
   createServer() {
     const server =  {
       address() {
diff --git a/src/connection.js b/src/connection.js
@@ -266,6 +266,7 @@ function Connection(options, queues = {}, { onopen = noop, onend = noop, onclose
     socket.removeAllListeners()
     socket = tls.connect({
       socket,
+      servername: net.isIP(socket.host) ? undefined : socket.host,
       ...(ssl === 'require' || ssl === 'allow' || ssl === 'prefer'
         ? { rejectUnauthorized: false }
         : ssl === 'verify-full'
