Skip to content

Commit

Permalink
test: fix hanging http obstext test
Browse files Browse the repository at this point in the history
Set the Connection header to 'close' to work around a v0.10 quirk.
Prevents the test from timing out due to a still open TCP connection.
Test introduced in 1e45a61 ("deps: update http-parser to version 1.2").

PR-URL: #5511
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
bnoordhuis committed Mar 2, 2016
1 parent 42ded2a commit e483f3f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/simple/test-http-header-obstext.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,9 @@ var server = http.createServer(common.mustCall(function(req, res) {
server.listen(common.PORT, function() {
http.get({
port: common.PORT,
headers: {'Test': 'Düsseldorf'}
headers: {'Connection': 'close', 'Test': 'Düsseldorf'}
}, common.mustCall(function(res) {
assert.equal(res.statusCode, 200);
server.close();
}));
});
});

0 comments on commit e483f3f

Please sign in to comment.