-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
deps: update undici to 7.11.0 #58859
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
Conversation
Review requested:
|
Tests are failing because nodejs/undici#4083 got fixed. Going to update the tests and remove the FIXME... |
`proxy-connection: keep-alive` is now correctly managed by undici after https://redirect.github.com/nodejs/undici/pull/4128
Looks like only the keep-alive part is fixed, the tunnling part is still a bit inconsistent with curl, left a comment in nodejs/undici#4180 (comment) and kept nodejs/undici#4083 open. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #58859 +/- ##
=======================================
Coverage 90.10% 90.10%
=======================================
Files 640 640
Lines 188493 188493
Branches 36971 36968 -3
=======================================
+ Hits 169843 169849 +6
+ Misses 11358 11351 -7
- Partials 7292 7293 +1 🚀 New features to boost your workflow:
|
33d5140
to
68d0b3f
Compare
This comment was marked as duplicate.
This comment was marked as duplicate.
68d0b3f
to
33d5140
Compare
This comment was marked as duplicate.
This comment was marked as duplicate.
Landed in 0b88e09 |
PR-URL: #58859 Co-authored-by: Joyee Cheung <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]>
This is an automated update of undici to 7.11.0.