diff options
author | Anatoli Papirovski <apapirovski@mac.com> | 2017-08-30 21:49:41 -0400 |
---|---|---|
committer | Ruben Bridgewater <ruben@bridgewater.de> | 2017-09-03 20:31:11 -0300 |
commit | e289540e12f7d9693ed180b5b2934aba1b5c62dd (patch) | |
tree | ee6031420e0449b47e21b9965496e91eaeca992a /test/parallel/test-http2-connect-method.js | |
parent | 198fcb9c620a0f08969d042f763cb491cb1c57d6 (diff) | |
download | android-node-v8-e289540e12f7d9693ed180b5b2934aba1b5c62dd.tar.gz android-node-v8-e289540e12f7d9693ed180b5b2934aba1b5c62dd.tar.bz2 android-node-v8-e289540e12f7d9693ed180b5b2934aba1b5c62dd.zip |
http2: adjust error types, test coverage
Change error types emitted from request and validatePriorityOptions
to be TypeError rather than the incorrect RangeError. Add tests
to confirm that all errors are thrown as expected (weight, parent,
exclusive, silent, endStream and getTrailers). Add test for method
CONNECT throwing error on missing :authority or superfluous :scheme
and :path.
PR-URL: https://github.com/nodejs/node/pull/15109
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Diffstat (limited to 'test/parallel/test-http2-connect-method.js')
-rw-r--r-- | test/parallel/test-http2-connect-method.js | 36 |
1 files changed, 36 insertions, 0 deletions
diff --git a/test/parallel/test-http2-connect-method.js b/test/parallel/test-http2-connect-method.js index 4cd8daf484..3626c7f2e5 100644 --- a/test/parallel/test-http2-connect-method.js +++ b/test/parallel/test-http2-connect-method.js @@ -12,6 +12,8 @@ const { URL } = require('url'); const { HTTP2_HEADER_METHOD, HTTP2_HEADER_AUTHORITY, + HTTP2_HEADER_SCHEME, + HTTP2_HEADER_PATH, NGHTTP2_CONNECT_ERROR } = http2.constants; @@ -53,6 +55,40 @@ server.listen(0, common.mustCall(() => { proxy.listen(0, () => { const client = http2.connect(`http://localhost:${proxy.address().port}`); + // confirm that :authority is required and :scheme & :path are forbidden + assert.throws( + () => client.request({ + [HTTP2_HEADER_METHOD]: 'CONNECT' + }), + common.expectsError({ + code: 'ERR_HTTP2_CONNECT_AUTHORITY', + message: ':authority header is required for CONNECT requests' + }) + ); + assert.throws( + () => client.request({ + [HTTP2_HEADER_METHOD]: 'CONNECT', + [HTTP2_HEADER_AUTHORITY]: `localhost:${port}`, + [HTTP2_HEADER_SCHEME]: 'http' + }), + common.expectsError({ + code: 'ERR_HTTP2_CONNECT_SCHEME', + message: 'The :scheme header is forbidden for CONNECT requests' + }) + ); + assert.throws( + () => client.request({ + [HTTP2_HEADER_METHOD]: 'CONNECT', + [HTTP2_HEADER_AUTHORITY]: `localhost:${port}`, + [HTTP2_HEADER_PATH]: '/' + }), + common.expectsError({ + code: 'ERR_HTTP2_CONNECT_PATH', + message: 'The :path header is forbidden for CONNECT requests' + }) + ); + + // valid CONNECT request const req = client.request({ [HTTP2_HEADER_METHOD]: 'CONNECT', [HTTP2_HEADER_AUTHORITY]: `localhost:${port}`, |