fix(ext/node): expose Http2Session and nghttp2ErrorString on http2 internalBinding (#33732)

Enables `test-http2-client-onconnect-errors` in node_compat suite.

Co-authored-by: divybot <divybot@users.noreply.github.com>
Co-authored-by: Divy Srivastava <me@littledivy.com>

Author: divybot

ext/node/polyfills/http2.ts

@@ -74,6 +74,7 @@ import {
   streamBaseState,
 } from "ext:deno_node/internal_binding/stream_wrap.ts";
 import {
+  Http2Session as BindingHttp2Session,
   Http2Stream as BindingHttp2Stream,
 } from "ext:deno_node/internal_binding/http2.ts";
 import { EventEmitter } from "node:events";
@@ -965,13 +966,17 @@ function requestOnConnect(headersList, options) {
 
   // `ret` will be either the reserved stream ID (if positive)
   // or an error code (if negative)
-  const ret = session[kHandle].request(
-    headersList[0],
-    headersList[1],
-    streamOptions,
-    options.parent | 0,
-    options.weight | 0,
-    !!options.exclusive,
+  const ret = ReflectApply(
+    BindingHttp2Session.prototype.request,
+    session[kHandle],
+    [
+      headersList[0],
+      headersList[1],
+      streamOptions,
+      options.parent | 0,
+      options.weight | 0,
+      !!options.exclusive,
+    ],
   );
 
   // In an error condition, one of three possible response codes will be

ext/node/polyfills/internal_binding/http2.ts

@@ -18,6 +18,31 @@ class Http2Stream {
   }
 }
 
+class Http2Session {
+  request(
+    this: {
+      request(
+        headers: string,
+        count: number,
+        options: number,
+        parent: number,
+        weight: number,
+        exclusive: boolean,
+      ): unknown;
+    },
+    headers: string,
+    count: number,
+    options: number,
+    parent: number,
+    weight: number,
+    exclusive: boolean,
+  ): unknown {
+    // Tests replace this prototype method; otherwise `this` is the native
+    // handle, so dispatch falls through to the handle's own `request` op.
+    return this.request(headers, count, options, parent, weight, exclusive);
+  }
+}
+
 function nghttp2ErrorString(integerCode: number) {
   return op_http2_error_string(integerCode);
 }

tests/node_compat/config.jsonc

@@ -1617,6 +1617,7 @@
     "parallel/test-http2-clean-output.js": {},
     "parallel/test-http2-client-connection-tunnelling.js": {},
     "parallel/test-http2-client-destroy.js": {},
+    "parallel/test-http2-client-onconnect-errors.js": {},
     "parallel/test-http2-client-port-80.js": {},
     "parallel/test-http2-client-priority-before-connect.js": {},
     "parallel/test-http2-client-promisify-connect-error.js": {},