fix: bad client destroy on servername change

Fixes: #3065
nodejs · Apr 7, 2024 · 40e866f · 40e866f
1 parent cae5625
commit 40e866f
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/lib/dispatcher/client.js b/lib/dispatcher/client.js
@@ -581,7 +581,15 @@ function _resume (client, sync) {
       }
 
       client[kServerName] = request.servername
-      client[kHTTPContext]?.destroy(new InformationalError('servername changed'))
+      client[kHTTPContext]?.destroy(new InformationalError('servername changed'), (err) => {
+        if (err) {
+          // TODO (fix): What to do?
+        }
+
+        client[kHTTPContext] = null
+
+        resume(client)
+      })
     }
 
     if (client[kConnecting]) {