fix(node): fixed https connection issue on Node.js 12 and 13

by applying the workaround as in nodejs/node#33343

fix #104

lib/context-https.ts

@@ -4,6 +4,10 @@ import { connect, ConnectionOptions, TLSSocket } from "tls";
 import { HttpProtocols } from "./core";
 import { AltNameMatch, parseOrigin } from "./san";
 
+
+const needsSocketHack = [ "12", "13" ]
+	.includes( process.versions.node.split( '.' )[ 0 ] );
+
 const alpnProtocols =
 {
 	http1: Buffer.from( "\x08http/1.1" ),
@@ -84,6 +88,12 @@ export function connectTLS(
 			resolve( { socket, protocol, altNameMatch } );
 		} );
 
+		if ( needsSocketHack )
+			socket.once( 'secureConnect', ( ) =>
+			{
+				( socket as any ).secureConnecting = false;
+			} );
+
 		socket.once( "error", reject );
 	} );
 }

lib/fetch-http2.ts

@@ -248,8 +248,7 @@ async function fetchImpl(
 				}
 			) );
 
-			stream.on( "response", guard(
-				( headers: IncomingHttp2Headers ) =>
+			stream.on( "response", guard( ( headers: IncomingHttp2Headers ) =>
 			{
 				const {
 					signal: bodySignal = void 0,

test/fetch-h2/context.ts

@@ -131,7 +131,7 @@ describe( `context (${version} over ${proto.replace( ":", "" )})`, ( ) =>
 				).toBeTruthy( );
 			}
 
-			disconnectAll( );
+			await disconnectAll( );
 
 			await server.shutdown( );
 		} );
@@ -154,7 +154,7 @@ describe( `context (${version} over ${proto.replace( ":", "" )})`, ( ) =>
 			const res = await response.json( );
 			expect( res[ "user-agent" ] ).toBe( "foobar" );
 
-			disconnectAll( );
+			await disconnectAll( );
 
 			await server.shutdown( );
 		} );

test/fetch-h2/index.ts

@@ -824,8 +824,8 @@ describe( `premature stream close (${protoVersion})`, ( ) =>
 			const expected =
 				version === "http1"
 				? "socket hang up"
-				: "Stream prematurely closed";
-			expect( err.message ).toContain( expected );
+				: [ "Stream prematurely closed", "Connection closed" ];
+			expect( expected ).toContain( err.message );
 		}
 
 		await disconnectAll( );