Skip to content

Log whenever AbortSignal is used over RPC#4103

Merged
npaun merged 2 commits intomainfrom
npaun/abortsignal-rpc-log
May 7, 2025
Merged

Log whenever AbortSignal is used over RPC#4103
npaun merged 2 commits intomainfrom
npaun/abortsignal-rpc-log

Conversation

@npaun
Copy link
Member

@npaun npaun commented May 7, 2025

We're going to have to put AbortSignal RPC support behind a compat flag ASAP, because it spuriously causes the outcome of a request to show exception even though things work fine. First we need to make sure no one is using this feature.

@npaun npaun requested review from danlapid, jasnell and kentonv May 7, 2025 18:06
@npaun npaun requested review from a team as code owners May 7, 2025 18:06
@npaun npaun force-pushed the npaun/abortsignal-rpc-log branch from 7503dc2 to 8962a0c Compare May 7, 2025 18:21
@npaun npaun enabled auto-merge (squash) May 7, 2025 18:27
Co-authored-by: Kenton Varda <kenton@cloudflare.com>
@npaun npaun merged commit 6679b07 into main May 7, 2025
18 checks passed
@npaun npaun deleted the npaun/abortsignal-rpc-log branch May 7, 2025 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants