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

HTTP concepts of error codes #4352

Closed
LPardue opened this issue Nov 15, 2020 · 1 comment · Fixed by #4384
Closed

HTTP concepts of error codes #4352

LPardue opened this issue Nov 15, 2020 · 1 comment · Fixed by #4384
Labels
-http ietf-lc An issue that was raised during IETF Last Call.
Milestone

Comments

@LPardue
Copy link
Member

LPardue commented Nov 15, 2020

In the HTTP genart review, @theri writes:

The concepts around error codes is not entirely clear to me, e.g.,
H3_REQUEST_INCOMPLETE or H3_NO_ERROR - Are these a separate concept from HTTP
response status codes such as "200 OK" and "404 Not Found", etc.? Are these
error codes related to QUIC errors? Is the "application error code" mentioned
in Section 5.3 the same concept or is it a different concept? Is the
"application error code" an HTTP/3 concept of a QUIC concept? Is "connection
error" used in Section 7 the same concept or is it a difference concept? Does a
"connection error" as used in Section 7 always mean that the entire connection
fails, as opposed to a single stream? Section 8 aims to explain some of these
concepts, and I think the document should reference to this section for more
information when it starts mentioning all these different error types. (Or, if
they are all the same, perhaps the terminology should be unified.)

@LPardue LPardue added -http ietf-lc An issue that was raised during IETF Last Call. labels Nov 15, 2020
@LPardue LPardue added this to the http-genart milestone Nov 15, 2020
@MikeBishop
Copy link
Contributor

As you note, Section 8 describes the types of error, but mostly defers to QUIC-TRANSPORT for that. It's probably worth pulling some more context into this document.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-http ietf-lc An issue that was raised during IETF Last Call.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants