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.

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

terminology/definition of where request target and response status live #372

Closed
reschke opened this issue May 22, 2020 · 4 comments · Fixed by #397
Closed

terminology/definition of where request target and response status live #372

reschke opened this issue May 22, 2020 · 4 comments · Fixed by #397

Comments

@reschke
Copy link
Contributor

reschke commented May 22, 2020

Section 2.1 has:

A client sends an HTTP request to a server in the form of a request message, beginning with a method (Section 7) and URI, followed by header fields containing request modifiers, client information, and representation metadata (Section 4), and finally a payload body (if any, Section 6.3.3).

A server responds to a client's request by sending one or more HTTP response messages, each beginning with a success or error code (Section 9), possibly followed by header fields containing server information, resource metadata, and representation metadata (Section 4), and finally a payload body (if any, Section 6.3.3).

So this is also specific to 1.1.

My proposal would be to define terms for request-line and status-line that are indenpendant of the protocol version.

For 1.1, these would map to what we had before (and might have to include the "Host" header field, right?).

For 2 (and 3, I guess), they would map to the pseudo header fields (all?).

We'd also have to fix the text above, as some ("Host") or all (pseudo header fields) are actually contained in the header field section.

@royfielding
Copy link
Member

I think we just changed this section. Does this issue still apply?

@mnot
Copy link
Member

mnot commented Jun 3, 2020

I think this is an unnecessary level of abstraction; it just adds to the terms and concepts that people have to remember, and isn't as familiar as just saying "method and request target" and "status code and optional status phrase".

@mnot
Copy link
Member

mnot commented Jul 2, 2020

Discussed; issue is that in 2.1 Client/Server, instances of "beginning with..." are misleading.

@royfielding
Copy link
Member

currently says

A client sends an HTTP request to a server in the form of a request message, beginning with a method (Section 7) and request target, followed by header fields containing request modifiers, client information, and representation metadata (Section 4), and finally a payload body (if any, Section 6.3.3).

@mnot mnot self-assigned this Jul 2, 2020
mnot added a commit that referenced this issue Jul 8, 2020
reschke added a commit that referenced this issue Jul 10, 2020
… proposed text (#372)Use Roy's proposed text (#372)Use Roy's proposed text (#372)Use Roy's proposed text (#372)Use Roy's proposed text (#372)
reschke added a commit that referenced this issue Jul 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

3 participants