4cd8e1914a
This also notably changes error handling for request(). Previously some invalid header values (but not all) would cause the session to be unnecessarily destroyed automatically, e.g. passing an unparseable header name to request(). This is no longer the case: header validation failures will throw an error, but will not destroy the session or emit 'error' events. PR-URL: https://github.com/nodejs/node/pull/57917 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
60 lines
1.5 KiB
JavaScript
60 lines
1.5 KiB
JavaScript
'use strict';
|
|
const common = require('../common');
|
|
if (!common.hasCrypto) { common.skip('missing crypto'); }
|
|
const assert = require('assert');
|
|
const http2 = require('http2');
|
|
|
|
const server1 = http2.createServer();
|
|
|
|
server1.listen(0, common.mustCall(() => {
|
|
const session = http2.connect(`http://localhost:${server1.address().port}`);
|
|
// Check for req headers
|
|
assert.throws(() => {
|
|
session.request({ 'no underscore': 123 });
|
|
}, {
|
|
code: 'ERR_INVALID_HTTP_TOKEN'
|
|
});
|
|
session.close();
|
|
server1.close();
|
|
}));
|
|
|
|
const server2 = http2.createServer(common.mustCall((req, res) => {
|
|
// check for setHeader
|
|
assert.throws(() => {
|
|
res.setHeader('x y z', 123);
|
|
}, {
|
|
code: 'ERR_INVALID_HTTP_TOKEN'
|
|
});
|
|
res.end();
|
|
}));
|
|
|
|
server2.listen(0, common.mustCall(() => {
|
|
const session = http2.connect(`http://localhost:${server2.address().port}`);
|
|
const req = session.request();
|
|
req.on('end', common.mustCall(() => {
|
|
session.close();
|
|
server2.close();
|
|
}));
|
|
}));
|
|
|
|
const server3 = http2.createServer(common.mustCall((req, res) => {
|
|
// check for writeHead
|
|
assert.throws(common.mustCall(() => {
|
|
res.writeHead(200, {
|
|
'an invalid header': 123
|
|
});
|
|
}), {
|
|
code: 'ERR_INVALID_HTTP_TOKEN'
|
|
});
|
|
res.end();
|
|
}));
|
|
|
|
server3.listen(0, common.mustCall(() => {
|
|
const session = http2.connect(`http://localhost:${server3.address().port}`);
|
|
const req = session.request();
|
|
req.on('end', common.mustCall(() => {
|
|
server3.close();
|
|
session.close();
|
|
}));
|
|
}));
|