Title: Node 10.14.x Regression in parsing of binary upgrade response body · Issue #24958 · nodejs/node · GitHub
Open Graph Title: Node 10.14.x Regression in parsing of binary upgrade response body · Issue #24958 · nodejs/node
X Title: Node 10.14.x Regression in parsing of binary upgrade response body · Issue #24958 · nodejs/node
Description: Version: 10.14.1 Platform: MacOS 10.13.2 Subsystem: (Docker > ubuntu:xenial) It appears that the v10.14.x version range introduces a regression which was resolved by #17806. This gist can consistently reproduce the error: https://gist.gi...
Open Graph Description: Version: 10.14.1 Platform: MacOS 10.13.2 Subsystem: (Docker > ubuntu:xenial) It appears that the v10.14.x version range introduces a regression which was resolved by #17806. This gist can consisten...
X Description: Version: 10.14.1 Platform: MacOS 10.13.2 Subsystem: (Docker > ubuntu:xenial) It appears that the v10.14.x version range introduces a regression which was resolved by #17806. This gist can consis...
Opengraph URL: https://github.com/nodejs/node/issues/24958
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Node 10.14.x Regression in parsing of binary upgrade response body","articleBody":"\u003c!--\r\nThank you for reporting a possible bug in Node.js.\r\n\r\nPlease fill in as much of the template below as you can.\r\n\r\nVersion: output of `node -v`\r\nPlatform: output of `uname -a` (UNIX), or version and 32 or 64-bit (Windows)\r\nSubsystem: if known, please specify the affected core module name\r\n\r\nIf possible, please provide code that demonstrates the problem, keeping it as\r\nsimple and free of external dependencies as you can.\r\n--\u003e\r\n\r\n* **Version**: 10.14.1\r\n* **Platform**: MacOS 10.13.2\r\n* **Subsystem**: _(Docker \u003e ubuntu:xenial)_\r\n\r\nIt appears that the v10.14.x version range introduces a regression which was resolved by #17806. This gist can consistently reproduce the error: https://gist.github.com/shellscape/2020916f92be5d61f4d411fb6d2bce61\r\n\r\nNote that stepping back down to any 10.x version lower than 10.14.0 does not reproduce the error. We've spun up brand new containers to verify the issue was isolated to 10.14.x. The same containers (config) from scratch with 10.13.x are completely fine. And have verified this on three different machines (both Windows and Mac OS), in different geographic locations (U.S., Brazil, New Zealand).\r\n\r\nThis was noticed with use of the [websockets/ws](/websockets/ws) module. The immediate error stack produced:\r\n\r\n```\r\n RangeError: Invalid WebSocket frame: RSV1 must be clear\r\n\r\n Receiver.getInfo (node_modules/ws/lib/receiver.js:168:14)\r\n Receiver.startLoop (node_modules/ws/lib/receiver.js:121:22)\r\n Receiver._write (node_modules/ws/lib/receiver.js:69:10)\r\n Socket.socketOnData (node_modules/ws/lib/websocket.js:816:35)\r\n```\r\n\r\nAnd the subsequent stack produced is:\r\n\r\n```\r\nError [ERR_STREAM_WRITE_AFTER_END]: write after end\r\n at writeAfterEnd (_stream_writable.js:243:12)\r\n at Socket.Writable.write (_stream_writable.js:291:5)\r\n at Sender.sendFrame (/node_modules/ws/lib/sender.js:404:20)\r\n at Sender.doClose (/node_modules/ws/lib/sender.js:141:10)\r\n at Sender.close (/node_modules/ws/lib/sender.js:128:12)\r\n at WebSocket.close (/node_modules/ws/lib/websocket.js:215:18)\r\n at Receiver.receiverOnConclude (/node_modules/ws/lib/websocket.js:695:32)\r\n at Receiver.emit (events.js:182:13)\r\n at Receiver.controlMessage (/node_modules/ws/lib/receiver.js:436:14)\r\n at Receiver.getData (/node_modules/ws/lib/receiver.js:330:42)\r\n```\r\n\r\nBoth, when traced to the best of my ability, point back to the same problem as described in #17806.","author":{"url":"https://github.com/shellscape","@type":"Person","name":"shellscape"},"datePublished":"2018-12-11T14:52:12.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":14},"url":"https://github.com/24958/node/issues/24958"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:7aa3bd24-72bd-6742-e9e8-690fef14d597 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | E86C:2E7285:302F633:41F49C6:6A4EFEA0 |
| html-safe-nonce | 76b35ed7bc2656631d66d83d0a9d3e709830c97e8f5bde0d03d38cdafb79bcac |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFODZDOjJFNzI4NTozMDJGNjMzOjQxRjQ5QzY6NkE0RUZFQTAiLCJ2aXNpdG9yX2lkIjoiNjA0NTQ1NzkzMTY3OTIzNTc0NCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 46742fa1e32b933cd27fc2ebc997e37594d9b3b80fa0391835b5bdd260d05420 |
| hovercard-subject-tag | issue:389800858 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/nodejs/node/24958/issue_layout |
| twitter:image | https://opengraph.githubassets.com/fd07baebf4e51d1113bcceef86f7342d8c80d960c4bbcab1ace7cbbe480ab395/nodejs/node/issues/24958 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/fd07baebf4e51d1113bcceef86f7342d8c80d960c4bbcab1ace7cbbe480ab395/nodejs/node/issues/24958 |
| og:image:alt | Version: 10.14.1 Platform: MacOS 10.13.2 Subsystem: (Docker > ubuntu:xenial) It appears that the v10.14.x version range introduces a regression which was resolved by #17806. This gist can consisten... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | shellscape |
| hostname | github.com |
| expected-hostname | github.com |
| None | b92d11c0aa4a77d54ef4af1078b6a15fb5a70a215b30c4ecf28889d5a8e656d9 |
| turbo-cache-control | no-preview |
| go-import | github.com/nodejs/node git https://github.com/nodejs/node.git |
| octolytics-dimension-user_id | 9950313 |
| octolytics-dimension-user_login | nodejs |
| octolytics-dimension-repository_id | 27193779 |
| octolytics-dimension-repository_nwo | nodejs/node |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 27193779 |
| octolytics-dimension-repository_network_root_nwo | nodejs/node |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 2b8f23afb982271f1b22258a94aede67a6b77760 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width