Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

http2: add compat trailers, adjust multi-headers #15193

Closed
wants to merge 3 commits into from
Closed

http2: add compat trailers, adjust multi-headers #15193

wants to merge 3 commits into from

Commits on Sep 5, 2017

  1. 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.

Refs: expressjs/express#3390 (comment)
    @apapirovski
    apapirovski committed Sep 5, 2017
    Copy the full SHA
    7ac07be View commit details
    Browse the repository at this point in the history

  2. Add proper support for raw headers & trailers, incorporate feedback

    @apapirovski
    apapirovski committed Sep 5, 2017
    Copy the full SHA
    52e79ea View commit details
    Browse the repository at this point in the history

  3. End readable side of the stream on trailers or bodyless requests

    @apapirovski
    apapirovski committed Sep 5, 2017
    Copy the full SHA
    85f9ab0 View commit details
    Browse the repository at this point in the history