perry-ext-http: https.createServer 'upgrade' listeners never fire #11159

https.createServer 'upgrade' listeners never fire

upgrade.rs's fire_upgrade_listeners resolves the server only as get_handle_mut::<HttpServer>. The turnloop connection layer decides an upgrade has a listener through with_base_server, which accepts HttpsServer too. It hands the connection to perry-ext-net and queues the upgrade, and drain_upgrades runs for HTTPS handles as well. The final dispatch then returns early for an HttpsServer, so:

• the listener never runs, and
• the adopted socket (and the request handle) are never handed to user code.

This code predates #11144 (unchanged at b75e6a6eb). CodeRabbit found it while reviewing #11144.

Repro (Linux, perrymaster, Node 26.5.1). Perry built from #11144 head c1f36c25b. The certificate is the inline CN=localhost fixture from test-files/test_gap_turnloop_https_server.ts (elided here).

import https from 'node:https';
import tls from 'node:tls';
// const CERT = ..., KEY = ...  (inline CN=localhost fixture from test_gap_turnloop_https_server.ts)
const server: any = https.createServer({ key: KEY, cert: CERT }, (_req: any, res: any) => res.end('plain'));
server.on('upgrade', (req: any, socket: any, _head: any) => {
  console.log('server upgrade event: ' + req.headers.upgrade);
  socket.end('HTTP/1.1 101 Switching Protocols
Connection: Upgrade
Upgrade: foo

');
});
server.listen(0, '127.0.0.1', () => {
  const s = tls.connect({ host: '127.0.0.1', port: server.address().port, rejectUnauthorized: false }, () => {
    s.write('GET / HTTP/1.1
Host: localhost
Connection: Upgrade
Upgrade: foo

');
  });
  let got = '';
  s.on('data', (d: any) => { got += d.toString(); });
  const done = () => { console.log('client got: ' + (got.split('
')[0] || '<nothing>')); server.close(); process.exit(0); };
  s.on('end', done);
  setTimeout(done, 1500);
});

== node
server upgrade event: foo
client got: HTTP/1.1 101 Switching Protocols
== perry
client got: <nothing>

Fix direction: resolve the base server the way with_base_server does (HttpServer | HttpsServer) in fire_upgrade_listeners. When no listener runs, release the adopted socket.