Skip to content

Commit fece5e2

Browse files
author
Arthur Gautier
committed
test: remoteAddress may be undefined
see nodejs/node#4198 test case: ``` javascript var net = require('net'); var server = net.createServer(function(c) { //'connection' listener c.on('end', function() { console.log(c.remoteAddress); server.close(); }); }); server.listen(8124, function() { var client = net.connect({port: 8124}, function(c) { client.end(); }); }); ``` Signed-off-by: Arthur Gautier <[email protected]>
1 parent d125b19 commit fece5e2

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

test/test.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,13 @@ describe('forwarded(req)', function () {
2626
})
2727
assert.deepEqual(forwarded(req), ['127.0.0.1', '10.0.0.1', '10.0.0.2'])
2828
})
29+
30+
it('should parse headers with disconnected client', function () {
31+
var req = createReq(undefined, {
32+
'x-forwarded-for': '10.0.0.2,, 10.0.0.1'
33+
})
34+
assert.deepEqual(forwarded(req), ['10.0.0.1', '10.0.0.2'])
35+
})
2936
})
3037

3138
function createReq(socketAddr, headers) {

0 commit comments

Comments
 (0)