You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Nov 6, 2022. It is now read-only.
@bnoordhuis There is no http request body for the http request with HEAD method. And I found that the parser can't be finished normally (on_message_complete callbacks is triggered) in this situation.
This change matches a HEAD request (which had Upgrade headers or for which the user returned 2 from the on_headers_complete callback). A HEAD response may have a Content-Length but does not include the body. A HEAD request does not have the same rule. It really should not have a body, but if it has a non-zero Content-Length, the body must be present (and should be ignored by the server).
A payload within a HEAD request message has no defined semantics;
sending a payload body on a HEAD request might cause some existing
implementations to reject the request.
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Here is the diff code:
The text was updated successfully, but these errors were encountered: