Skip to content
This repository has been archived by the owner on Oct 16, 2021. It is now read-only.

Commit

Permalink
test: check that --insecure-http-parser works
Browse files Browse the repository at this point in the history
Backport ab1fcb8

Original commit message:

    Test that using --insecure-http-parser will disable validation of
    invalid characters in HTTP headers.

    See:
    - nodejs/node#30567

    PR-URL: nodejs/node#31253
    Backport-PR-URL: nodejs/node#30473
    Reviewed-By: Richard Lau <riclau@uk.ibm.com>
    Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
  • Loading branch information
zsw007 committed Feb 12, 2020
1 parent 942e61e commit ec48296
Show file tree
Hide file tree
Showing 2 changed files with 70 additions and 0 deletions.
35 changes: 35 additions & 0 deletions test/parallel/test-http-insecure-parser-legacy.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
// Flags: --insecure-http-parser --http-parser=legacy

'use strict';
const common = require('../common');
const assert = require('assert');
const http = require('http');
const net = require('net');

const server = http.createServer(function(req, res) {
assert.strictEqual(req.headers['content-type'], 'text/te\bt');
req.pipe(res);
});

server.listen(0, common.mustCall(function() {
const bufs = [];
const client = net.connect(
this.address().port,
function() {
client.write(
'GET / HTTP/1.1\r\n' +
'Content-Type: text/te\x08t\r\n' +
'Connection: close\r\n\r\n');
}
);
client.on('data', function(chunk) {
bufs.push(chunk);
});
client.on('end', common.mustCall(function() {
const head = Buffer.concat(bufs)
.toString('latin1')
.split('\r\n')[0];
assert.strictEqual(head, 'HTTP/1.1 200 OK');
server.close();
}));
}));
35 changes: 35 additions & 0 deletions test/parallel/test-http-insecure-parser.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
// Flags: --insecure-http-parser

'use strict';
const common = require('../common');
const assert = require('assert');
const http = require('http');
const net = require('net');

const server = http.createServer(function(req, res) {
assert.strictEqual(req.headers['content-type'], 'text/te\bt');
req.pipe(res);
});

server.listen(0, common.mustCall(function() {
const bufs = [];
const client = net.connect(
this.address().port,
function() {
client.write(
'GET / HTTP/1.1\r\n' +
'Content-Type: text/te\x08t\r\n' +
'Connection: close\r\n\r\n');
}
);
client.on('data', function(chunk) {
bufs.push(chunk);
});
client.on('end', common.mustCall(function() {
const head = Buffer.concat(bufs)
.toString('latin1')
.split('\r\n')[0];
assert.strictEqual(head, 'HTTP/1.1 200 OK');
server.close();
}));
}));

0 comments on commit ec48296

Please sign in to comment.