http2: set origin name correctly when servername is empty

Fixes: #39919 Refs: #39934 PR-URL: #42838 Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Matteo Collina <[email protected]>

Author: ofirbarak

lib/internal/http2/core.js

@@ -3119,7 +3119,7 @@ function initializeTLSOptions(options, servername){
 options.ALPNProtocols=['h2'];
 if(options.allowHTTP1===true)
 ArrayPrototypePush(options.ALPNProtocols,'http/1.1');
-if(servername!==undefined&&options.servername===undefined)
+if(servername!==undefined&&!options.servername)
 options.servername=servername;
 returnoptions;
 }

test/parallel/test-http2-create-client-secure-session.js

@@ -91,3 +91,9 @@ verifySecureSession(
 loadKey('agent1-cert.pem'),
 loadKey('ca1-cert.pem'),
 {servername: 'agent1'});
+
+verifySecureSession(
+loadKey('agent8-key.pem'),
+loadKey('agent8-cert.pem'),
+loadKey('fake-startcom-root-cert.pem'),
+{servername: ''});