HTTP2: Add a space character after the status code #755

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@mkauf
Contributor
mkauf commented Apr 6, 2016

The space character after the status code is mandatory, even if the reason phrase is empty (see RFC 2616)

Status-Line = HTTP-Version SP Status-Code SP Reason-Phrase CRLF

I think that this change will improve the compatibility with existing applications that use libcurl.

@mkauf mkauf HTTP2: Add a space character after the status code
The space character after the status code is mandatory, even if the reason
phrase is empty (see RFC 2616)
0964d96
@bagder bagder added a commit that closed this pull request Apr 6, 2016
@mkauf @bagder mkauf + bagder HTTP2: Add a space character after the status code
The space character after the status code is mandatory, even if the
reason phrase is empty (see RFC 7230 section 3.1.2)

Closes #755
3a8e38d
@bagder bagder closed this in 3a8e38d Apr 6, 2016
@bagder
Member
bagder commented Apr 6, 2016

Thanks! I updated the RFC reference in the commit message... =)

@mkauf mkauf deleted the mkauf:http2_status_code_space branch Apr 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment