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

ABNF message format #62

Closed
reschke opened this issue Apr 30, 2018 · 2 comments
Closed

ABNF message format #62

reschke opened this issue Apr 30, 2018 · 2 comments
Assignees
@reschke
Labels
editorial h1-messaging

Comments

@reschke
Copy link
Contributor

reschke commented Apr 30, 2018

We currently have:

  HTTP-message   = start-line
                   *( header-field CRLF )
                   CRLF
                   [ message-body ]

Maybe it would be more consistent to remove CRLF from start-line, and then to have

  HTTP-message   = start-line CRLF
                   *( header-field CRLF )
                   CRLF
                   [ message-body ]
@mnot
Copy link
Member

mnot commented Jun 29, 2018

LGTM.

@annevk annevk mentioned this issue Oct 15, 2018
Closed
@reschke
Copy link
Contributor Author

reschke commented Mar 26, 2019

...waiting until #197 is resolved...

@reschke reschke self-assigned this Jul 6, 2019
reschke added a commit that referenced this issue Jul 6, 2019
@reschke
Move the trailing CRLF from the line grammars into the message format (…
2ffabe9 
…#62)
reschke added a commit that referenced this issue Jul 8, 2019
@reschke
fix typo (#62)
de32a9e
reschke added a commit that referenced this issue Jul 26, 2019
@reschke
Merge pull request #232 from httpwg/reschke-i62
65572c4 
Move the trailing CRLF from the line grammars into the message format (#62)
@reschke reschke closed this as completed Jul 26, 2019
@reschke reschke mentioned this issue Jan 22, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@reschke reschke

Labels
editorial h1-messaging
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mnot @reschke