Skip to content

http2: replace unreachable error with assertion

Rodrigo Muino Tomonari requested to merge github/fork/Trott/unreachable into master

"That particular emit('error', ...) is largely defensively coded and should not ever actually happen." Sounds like an assertion rather than an error event. The code in question has no test coverage because it is believed to be unreachable.

Fixes: https://github.com/nodejs/node/issues/20673

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Merge request reports