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: Verify if we need to get http client support for 100 Continue #101

Closed
harshavardhana opened this issue Jun 24, 2015 · 2 comments
Closed
Assignees

Comments

@harshavardhana
Copy link
Member

golang/go#3665 - waiting on this Go 1.6

@harshavardhana
Copy link
Member Author

8.2.3 Use of the 100 (Continue) Status

The purpose of the 100 (Continue) status (see section 10.1.1) is to allow a client that is sending a request message with a request body to determine if the origin server is willing to accept the request (based on the request headers) before the client sends the request body. In some cases, it might either be inappropriate or highly inefficient for the client to send the body if the server will reject the message without looking at the body.

Requirements for HTTP/1.1 clients:

  - If a client will wait for a 100 (Continue) response before
    sending the request body, it MUST send an Expect request-header
    field (section 14.20) with the "100-continue" expectation.
  - A client MUST NOT send an Expect request-header field (section
    14.20) with the "100-continue" expectation if it does not intend
    to send a request body.

@harshavardhana
Copy link
Member Author

To keep note of while implementing this feature - https://tools.ietf.org/html/rfc7231#page-35

      A client that receives a 417 (Expectation Failed) status code in
      response to a request containing a 100-continue expectation SHOULD
      repeat that request without a 100-continue expectation, since the
      417 response merely indicates that the response chain does not
      support expectations (e.g., it passes through an HTTP/1.0 server).

Currently being transparently being implemented by http.Client we need to make sure that Golang http.Client would behave this way.

@harshavardhana harshavardhana changed the title Verify if we need to get http client support for 100 Continue http: Verify if we need to get http client support for 100 Continue Feb 28, 2016
harshavardhana pushed a commit to harshavardhana/minio-go that referenced this issue Feb 29, 2016
harshavardhana pushed a commit to harshavardhana/minio-go that referenced this issue Feb 29, 2016
harshavardhana pushed a commit to harshavardhana/minio-go that referenced this issue Mar 4, 2016
ckemper67 pushed a commit to ckemper67/minio-go that referenced this issue Mar 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant