quic: refactor/improve QuicSocket ready event handling

PR-URL: #34247
Reviewed-By: Anna Henningsen <anna@addaleax.net>

Author: jasnell

lib/internal/quic/core.js

@@ -1031,7 +1031,13 @@ class QuicSocket extends EventEmitter {
     // used to either listen or connect. No QuicServerSession should
     // exist before this event, and all QuicClientSession will remain
     // in Initial states until ready is invoked.
-    process.nextTick(emit.bind(this, 'ready'));
+    process.nextTick(() => {
+      try {
+        this.emit('ready');
+      } catch (error) {
+        this.destroy(error);
+      }
+    });
   }
 
   // Called when a QuicEndpoint closes

test/parallel/test-quic-server-ready-event-error-async.js

@@ -0,0 +1,31 @@
+// Flags: --no-warnings
+'use strict';
+
+const common = require('../common');
+if (!common.hasQuic)
+  common.skip('missing quic');
+
+const assert = require('assert');
+const {
+  key,
+  cert,
+  ca,
+} = require('../common/quic');
+
+const { createQuicSocket } = require('net');
+
+const options = { key, cert, ca, alpn: 'zzz' };
+
+const server = createQuicSocket({ server: options });
+
+server.on('session', common.mustNotCall());
+
+server.listen();
+
+server.on('error', common.mustCall((error) => {
+  assert.strictEqual(error.message, 'boom');
+}));
+
+server.on('ready', common.mustCall(async () => {
+  throw new Error('boom');
+}));

test/parallel/test-quic-server-ready-event-error.js

@@ -0,0 +1,31 @@
+// Flags: --no-warnings
+'use strict';
+
+const common = require('../common');
+if (!common.hasQuic)
+  common.skip('missing quic');
+
+const assert = require('assert');
+const {
+  key,
+  cert,
+  ca,
+} = require('../common/quic');
+
+const { createQuicSocket } = require('net');
+
+const options = { key, cert, ca, alpn: 'zzz' };
+
+const server = createQuicSocket({ server: options });
+
+server.on('session', common.mustNotCall());
+
+server.listen();
+
+server.on('error', common.mustCall((error) => {
+  assert.strictEqual(error.message, 'boom');
+}));
+
+server.on('ready', common.mustCall(() => {
+  throw new Error('boom');
+}));