mirror of
https://github.com/zebrajr/node.git
synced 2025-12-07 12:20:50 +01:00
fc40b7de46
Backport-PR-URL: https://github.com/nodejs/node/pull/18050 Backport-PR-URL: https://github.com/nodejs/node/pull/20456 PR-URL: https://github.com/nodejs/node/pull/17406 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> This is a significant cleanup and refactoring of the cleanup/close/destroy logic for Http2Stream and Http2Session. There are significant changes here in the timing and ordering of cleanup logic, JS apis. and various related necessary edits.
96 lines
2.6 KiB
JavaScript
96 lines
2.6 KiB
JavaScript
'use strict';
|
|
|
|
const common = require('../common');
|
|
if (!common.hasCrypto)
|
|
common.skip('missing crypto');
|
|
const assert = require('assert');
|
|
const h2 = require('http2');
|
|
|
|
const server = h2.createServer();
|
|
|
|
// we use the lower-level API here
|
|
server.on('stream', common.mustCall(onStream));
|
|
|
|
const status101regex =
|
|
/^HTTP status code 101 \(Switching Protocols\) is forbidden in HTTP\/2$/;
|
|
const afterRespondregex =
|
|
/^Cannot specify additional headers after response initiated$/;
|
|
|
|
function onStream(stream, headers, flags) {
|
|
|
|
assert.throws(() => stream.additionalHeaders({ ':status': 201 }),
|
|
common.expectsError({
|
|
code: 'ERR_HTTP2_INVALID_INFO_STATUS',
|
|
type: RangeError,
|
|
message: /^Invalid informational status code: 201$/
|
|
}));
|
|
|
|
assert.throws(() => stream.additionalHeaders({ ':status': 101 }),
|
|
common.expectsError({
|
|
code: 'ERR_HTTP2_STATUS_101',
|
|
type: Error,
|
|
message: status101regex
|
|
}));
|
|
|
|
common.expectsError(
|
|
() => stream.additionalHeaders({ ':method': 'POST' }),
|
|
{
|
|
code: 'ERR_HTTP2_INVALID_PSEUDOHEADER',
|
|
type: Error,
|
|
message: '":method" is an invalid pseudoheader or is used incorrectly'
|
|
}
|
|
);
|
|
|
|
// Can send more than one
|
|
stream.additionalHeaders({ ':status': 100 });
|
|
stream.additionalHeaders({ ':status': 100 });
|
|
|
|
stream.respond({
|
|
'content-type': 'text/html',
|
|
':status': 200
|
|
});
|
|
|
|
assert.throws(() => stream.additionalHeaders({ abc: 123 }),
|
|
common.expectsError({
|
|
code: 'ERR_HTTP2_HEADERS_AFTER_RESPOND',
|
|
type: Error,
|
|
message: afterRespondregex
|
|
}));
|
|
|
|
stream.end('hello world');
|
|
}
|
|
|
|
server.listen(0);
|
|
|
|
server.on('listening', common.mustCall(() => {
|
|
|
|
const client = h2.connect(`http://localhost:${server.address().port}`);
|
|
|
|
const req = client.request({ ':path': '/' });
|
|
|
|
// The additionalHeaders method does not exist on client stream
|
|
assert.strictEqual(req.additionalHeaders, undefined);
|
|
|
|
// Additional informational headers
|
|
req.on('headers', common.mustCall((headers) => {
|
|
assert.notStrictEqual(headers, undefined);
|
|
assert.strictEqual(headers[':status'], 100);
|
|
}, 2));
|
|
|
|
// Response headers
|
|
req.on('response', common.mustCall((headers) => {
|
|
assert.notStrictEqual(headers, undefined);
|
|
assert.strictEqual(headers[':status'], 200);
|
|
assert.strictEqual(headers['content-type'], 'text/html');
|
|
}));
|
|
|
|
req.resume();
|
|
|
|
req.on('end', common.mustCall(() => {
|
|
server.close();
|
|
client.close();
|
|
}));
|
|
req.end();
|
|
|
|
}));
|