Question

如果存在连接/代理错误，我正在尝试重试请求。由于某些原因，我不断收到这个错误，无论重试请求重试请求都无法恢复：

    Post https://m.somewebsite.co.uk/api/di/34433: http: ContentLength=222  with Body length 0

我做错了吗？我的第一个怀疑是http.Request以某种方式被消耗，因此在接下来的尝试中它不再是好的。我应该管理副本吗？

func Post(URL string, form url.Values, cl *http.Client) ([]byte, error) {
    req, err := http.NewRequest("POST", URL, strings.NewReader(form.Encode()))
    if err != nil {
        log.Error(err)
        return nil, err
    }
    req.Header.Set("User-Agent", ua)
    req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
    rsp, err := do(cl, req)

    if err != nil {
        return nil, err
    }
    defer rsp.Body.Close()
    b, err := ioutil.ReadAll(rsp.Body)
    if err != nil {
        log.Error(err)
        return nil, err
    }

    return b, nil
}

func do(cl *http.Client, req *http.Request)(*http.Response, error){
    rsp, err := cl.Do(req)
    for i := 0; IsErrProxy(err); i++ {
        log.Errorf("Proxy is slow or down ")
        time.Sleep(6 * time.Second)
5t      rsp, err = cl.Do(&ncp)
        if err == nil{
            return rsp, nil
        }
        if i > 10 {

            return nil, fmt.Errorf("after %v tries error: %v", i, err)
        }
    }
    return rsp, err
}

Answer 1

问题是在第一次调用Do（）时，请求体被读到最后。在后续调用Do（）时，不会从响应正文中读取任何数据。

修复方法是在for循环中移动body读取器的创建。这要求也在for循环中创建请求。

func Post(URL string, form url.Values, cl *http.Client) ([]byte, error) {
  body := form.Encode()
  for i := 0; i < 10; i++ {
    req, err := http.NewRequest("POST", URL, strings.NewReader(body))
    if err != nil {
        log.Error(err)
        return nil, err
    }
    req.Header.Set("User-Agent", ua)
    req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
    rsp, err := cl.Do(req)
    if err == nil {
        defer rsp.Body.Close()
        b, err := ioutil.ReadAll(rsp.Body)
        if err != nil {
            log.Error(err)
            return nil, err
        }
        return b, nil
    }

    if !IsErrorProxy(err) {
        return nil, err
    }

    log.Errorf("Proxy is slow or down ")
    time.Sleep(6 * time.Second)
  }
  return nil, fmt.Errorf("after 10 tries error: %v", err)
}

Answer 2

让我们看看http.NewRequest做什么

rc, ok := body.(io.ReadCloser)
if !ok && body != nil {
    rc = ioutil.NopCloser(body)
}
// The host's colon:port should be normalized. See Issue 14836.
u.Host = removeEmptyPort(u.Host)
req := &Request{
    Method:     method,
    URL:        u,
    Proto:      "HTTP/1.1",
    ProtoMajor: 1,
    ProtoMinor: 1,
    Header:     make(Header),
    Body:       rc,
    Host:       u.Host,
}

body的类型为io.Reader，并由io.ReaderCloser转换为ioutil.NopCloser。正如@CeriseLimón所说，Request.Body已被阅读且流更接近，所以当你再次Do()时，Body Length为0。

因此，我们可以在调用Do之前重置Request.Body。

func Post(URL string, form url.Values, cl *http.Client) ([]byte, error) {
    requestBodyString := form.Encode()
    req, err := http.NewRequest("POST", URL, strings.NewReader(requestBodyString))
    // ...

    rsp, err := do(cl, req, requestBodyString)

    //...
    return b, nil
}

func do(cl *http.Client, req *http.Request, requestBodyString string)(*http.Response, error){
    rsp, err := cl.Do(req)
    for i := 0; IsErrProxy(err); i++ {
        log.Errorf("Proxy is slow or down ")
        time.Sleep(6 * time.Second)
        // reset Request.Body
        req.Body = ioutil.NopCloser(strings.NewReader(requestBodyString))
        rsp, err = cl.Do(&req)
        if err == nil{
            return rsp, nil
        }
        if i > 10 {

            return nil, fmt.Errorf("after %v tries error: %v", i, err)
        }
    }
    return rsp, err
}

Answer 3

我找到了一种无需每次都重新创建请求的方法。这是一个示例代码，它是@CeriseLimón的非常小的修改，与@Rambo的代码类似，因为它只创建一次请求：

func Post(URL string, data *bytes.Buffer, cl *http.Client) ([]byte, error) {
    var err error

    req, err := http.NewRequest("POST", URL, ioutil.NopCloser(data))
    if err != nil {
        log.Errorf("Unable to create the request: %v", err)
        return nil, err
    }
    req.Header.Set("User-Agent", ua)

    for i := 0; i < 10; i++ {
        rsp, err := cl.Do(req)
        if err == nil && rsp.StatusCode == 200 {
            defer rsp.Body.Close()
            b, err := ioutil.ReadAll(rsp.Body)
            if err != nil {
                fmt.Printf("Error: %v", err)
                return nil, err
            }
            return b, nil
        }

        log.Errorf("Proxy is slow or down %v", err)
        time.Sleep(1 * time.Second)
    }
    return nil, fmt.Errorf("after 10 tries error: %v", err)
}

func main() {
    client := http.Client{}
    data := []byte{0, 1, 2, 3, 4}
    Post("http://server/my/api/resource/", bytes.NewBuffer(data), &client)
}

net / http：http：ContentLength = 222，正文长度为0

3 个答案: