-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade AbortSignal & AbortController to latest from WebKit #10106
Conversation
Fixes #9977 Closes #10086 Thank you @lithdew for investigating and most of the fixes. This adds more of the changes we missed from WebKit into Bun like the ability to follow other signals Co-Authored-By: Kenta Iwasaki <[email protected]>
❌ @Jarred-Sumner 3 files with test failures on linux-x64-baseline: |
❌ @Jarred-Sumner 3 files with test failures on linux-x64: |
❌ @Jarred-Sumner 4 files with test failures on bun-darwin-aarch64:
|
❌ @Jarred-Sumner 12 files with test failures on bun-windows-x86_64-haswell
|
❌ @Jarred-Sumner 2 files with test failures on bun-darwin-x64:
|
Fixes #9977 Closes #10086 Thank you @lithdew for investigating and most of the fixes. This adds more of the changes we missed from WebKit into Bun like the ability to follow other signals Co-authored-by: Kenta Iwasaki <[email protected]>
I'm seeing a leaking issue using remix with
let me know if i can help debug in some way. I'm trying to change a few things if I can isolate where it is. Memory just grows like crazy as more requests come in. |
What does this PR do?
Fixes #9977
Closes #10086
Thank you @lithdew for investigating and most of the fixes. This adds more of the changes we missed from WebKit into Bun like the ability to follow other signals
1st: after 2,000 requests
2nd: after 10,000 requests
3rd: after 50,000 requests
How did you verify your code works?