Closed
Description
This is advance notice that one CI test in ws may fail when Node.js 27 Alpha is released.
Now confirmed also in Node.js 27 Alpha release with ws@8.22.0
Situation
Running citgm ws fails on Node.js 27 nightly (macos, ubuntu & windows)
| Item | Status |
|---|---|
| npm package | https://www.npmjs.com/package/ws |
| npm status | active |
| last published | 8.22.0 Sep 26, 2026 |
| weekly downloads | 224,959,738 |
| repo location | https://github.com/websockets/ws |
| repo last update to default branch | Sep 26, 2026 |
| repo status | active |
| repo CI status | runs in Node.js 10-26, macos, ubuntu & windows |
| lookup - skip | win32 |
| lookup maintainer 1 | einaros (dormant) |
| lookup maintainer 2 | 3rd-Eden (dormant) |
| lookup maintainer 3 | lpinca |
Repo CI https://github.com/websockets/ws/blob/master/.github/workflows/ci.yml runs in Node.js 10-26, macos, ubuntu & windows.
Steps to reproduce
On Ubuntu 26.04.1 LTS, with n Node.js version manager installed.
n nightly # for example nightly/27.0.0-nightly20261007aa1bf2cc37
npm install citgm@latest -g
git clone https://github.com/websockets/ws
cd ws
npm install
npm testLogs
1) WebSocket
Connection establishing
fails if the Upgrade header field value cannot be read:
Uncaught Error: Parse Error: Header overflow
at Socket.socketOnData (node:_http_client:842:22)
at Socket.emit (node:events:514:20)
at addChunk (node:internal/streams/readable:605:12)
at readableAddChunkPushByteMode (node:internal/streams/readable:556:3)
at Readable.push (node:internal/streams/readable:436:5)
at TCP.onStreamRead (node:internal/stream_base_commons:194:23)Result
Passes in all current release lines & major operating systems:
| Node.js | npm | macos | ubuntu | windows |
|---|---|---|---|---|
| 22.23.3 | 10.9.9 | PASS | PASS | PASS |
| 24.21.0 | 11.19.0 | PASS | PASS | PASS |
| 26.10.0 | 11.19.1 | PASS | PASS | PASS |
Fails in current nightly-v27.0.0-nightly202610086d5e309871
nodejs/node#65010 introduces HPE_HEADER_OVERFLOW. If this commit is released with Node.js 27, then tests can be expected to fail.
References
Lines 470 to 474 in 4d3d256
| "ws": { | |
| "expectFail": "fips", | |
| "maintainers": ["einaros", "3rd-Eden", "lpinca"], | |
| "scripts": ["test -- --timeout 30000"] | |
| }, |
cc: @lpinca
Metadata
Metadata
Assignees
Labels
No labels
Activity
MikeMcC399 commentedon Oct 8, 2026
Additionally, according to websockets/ws#2330 @einaros & @3rd-Eden are no longer active in this repo. Probably they should be removed as maintainers for
wsin lookup.jsonlpinca commentedon Oct 8, 2026
Yes, this is the same as websockets/ws@2eb3be0 but for nodejs/node@e5778f7. I will fix it. Thanks for the ping.
MikeMcC399 commentedon Oct 13, 2026
Closing, as Node.js nightly builds are an unreliable metric. This was advance notice only.
websockets/ws#2340 has been opened, so it is already tracked in the
wsrepo.MikeMcC399 commentedon Oct 30, 2026
Fails in
v27.0.0-alpha.0.0.0lpinca commentedon Oct 30, 2026
I'll try to cut a new ws release today.
[-]`ws` fails on Node.js 27 nightly[/-][+]`ws` fails on Node.js 27[/+]MikeMcC399 commentedon Oct 30, 2026
Re-opening for tracking purposes. Will close again if tests are successful after new release.
lpinca commentedon Oct 30, 2026
Postponed to tomorrow, I'm out of time.
lpinca commentedon Oct 31, 2026
ws@8.22.1has been published to npm.MikeMcC399 commentedon Oct 31, 2026
Thanks very much @lpinca !
I've successfully tested in citgm running under GitHub Actions with the full matrix of:
actions/setup-node)So I'll close this issue now.