deps: update libuv to 1.53.0 - #66282
nodejs-github-bot wants to merge 1 commit into
Conversation
|
Review requested:
|
|
This should also fix the SmartOS Switching V8's entropy source to |
|
https://github.com/nodejs/node/actions/runs/36140009857/job/108087238784?pr=66282#step:6:4886 |
|
98c9628 to
d44f17a
Compare
|
This release includes libuv/libuv#5181, which fixes |
|
Hi, I looked into the two CI failures on this PR and have fixes on a branch, in case they're useful: Windows link failure. libuv 1.53's pipe code now calls WaitOnAddress / WakeByAddressSingle, which live in Synchronization.lib. Node's uv.gyp doesn't link that library, so the build fails with LNK2019 for both symbols (from libuv.lib(pipe.obj)), then LNK1120 in node_js2c.exe. I reproduced this at d44f17a with VS 17.14 / MSVC 19.44. test-blocklist failure on Linux. libuv 1.53 includes the change from 6179e7af, which replaces the hard-coded IPv6 address buffer size in uv_ip6_addr() with INET6_ADDRSTRLEN. Longer zoned IPv6 address parts are therefore parsed instead of being truncated at the old limit. FastCheckString() still rejects address parts of 40+ characters, so the fast and slow BlockList paths disagree for the same string. That's the test-blocklist.js:891 failure. I reproduced it on Fedora 44 / GCC 16.2. I used a closed-source coding agent to help investigate and draft these. I reviewed the changes, reproduced both failures and verified the fixes myself, and I'm happy to explain or adjust anything. Feel free to cherry-pick or adapt them into this PR, or let me know if you'd rather have a separate PR. |
This is an automated update of libuv to 1.53.0.