diff options
| author | Anatoli Papirovski <apapirovski@mac.com> | 2017-09-04 22:29:59 -0400 |
|---|---|---|
| committer | Matteo Collina <hello@matteocollina.com> | 2017-09-07 18:43:55 +0200 |
| commit | 641646463ddac7f306730a216675428e2bbe4dbf (patch) | |
| tree | 21893093eced2df5ef3c55499938739d9319de3d /test/parallel/test-http2-compat-serverresponse-statusmessage-property.js | |
| parent | 447715543b5f80cbee6496657179f1393a631fbf (diff) | |
| download | android-node-v8-641646463ddac7f306730a216675428e2bbe4dbf.tar.gz android-node-v8-641646463ddac7f306730a216675428e2bbe4dbf.tar.bz2 android-node-v8-641646463ddac7f306730a216675428e2bbe4dbf.zip | |
http2: add compat trailers, adjust multi-headers
Adds Http2ServerRequest trailers & rawTrailers functionality. Also fixes
behaviour of multi-headers to conform with the spec (all values but
set-cookie and cookie should be comma delimited, cookie should be
semi-colon delimited and only set-cookie should be an array). Adds
setter for statusMessage that warns, for backwards compatibility.
End readable side of the stream on trailers or bodyless requests
Refs: https://github.com/expressjs/express/pull/3390#discussion_r136718729
PR-URL: https://github.com/nodejs/node/pull/15193
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Diffstat (limited to 'test/parallel/test-http2-compat-serverresponse-statusmessage-property.js')
| -rw-r--r-- | test/parallel/test-http2-compat-serverresponse-statusmessage-property.js | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/test/parallel/test-http2-compat-serverresponse-statusmessage-property.js b/test/parallel/test-http2-compat-serverresponse-statusmessage-property.js index 782c239f0f..3b3ef6bdc2 100644 --- a/test/parallel/test-http2-compat-serverresponse-statusmessage-property.js +++ b/test/parallel/test-http2-compat-serverresponse-statusmessage-property.js @@ -23,6 +23,7 @@ server.listen(0, common.mustCall(function() { server.once('request', common.mustCall(function(request, response) { response.on('finish', common.mustCall(function() { assert.strictEqual(response.statusMessage, ''); + assert.strictEqual(response.statusMessage, ''); // only warn once server.close(); })); response.end(); |