Skip to content
Commit Detail

Commit d679549

Author
Harris Hancock <harris@cloudflare.com> 2026-04-07 17:55:56 +0100
Parents
696113e
Tree
a30f33c
EW-10529 Route OutgoingFactory subrequests through getSubrequestNoChecks

TcpPortOutgoingFactory (containers) and StreamOutgoingFactory (TCP
sockets) constructed WorkerInterface instances directly, bypassing
getSubrequestNoChecks() and missing the external memory adjustment
that pressures V8's GC. Route them through getSubrequestNoChecks() so
they get the same accounting as other subrequest types.

Also add comments on OutgoingFactory, CrossContextOutgoingFactory,
getSubrequestNoChecks(), getCapnpChannel(), and getClientWithTracing()
documenting the contract: implementations that create HTTP connections
should route through getSubrequestNoChecks() internally.

Assisted-by: Claude

Files changed

5 files changed~5 modified