mirror of https://github.com/nodejs/node.git
test: add ALPNProtocols option to clientOptions
Without this, the session is destroyed with the following error ``` Error [ERR_HTTP2_ERROR]: Protocol error at Http2Session.onSessionInternalError (internal/http2/core.js:756:26) Emitted 'error' event on ClientHttp2Session instance at: at emitClose (internal/http2/core.js:1010:10) at internal/http2/core.js:1048:7 at finish (internal/streams/writable.js:731:5) at processTicksAndRejections (internal/process/task_queues.js:80:21) { code: 'ERR_HTTP2_ERROR', errno: -505 } ``` The test then calls `session.close()` which tries to write to a destroyed socket. As a result, an unhandled `ECONNRESET` error is emitted in the v12 release line. PR-URL: https://github.com/nodejs/node/pull/35482 Refs: https://github.com/nodejs/node/pull/34859 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com>pull/35482/head
parent
1d5fa88eb8
commit
b474901438
|
@ -95,6 +95,7 @@ const { connect: tlsConnect } = require('tls');
|
|||
};
|
||||
|
||||
const clientOptions = {
|
||||
ALPNProtocols: ['h2'],
|
||||
port,
|
||||
rejectUnauthorized: false
|
||||
};
|
||||
|
|
Loading…
Reference in New Issue