dont rely on res
This commit is contained in:
parent
e0189b1f33
commit
e8503541c9
@ -121,11 +121,11 @@ func (c *Client) doWithRedirects(cli *http.Client, req *http.Request, via []*htt
|
||||
|
||||
res, err := cli.Do(req)
|
||||
if err != nil {
|
||||
c.traceResponse(tracedReq, nil)
|
||||
c.traceResponse(req, tracedReq, nil)
|
||||
return res, err
|
||||
}
|
||||
|
||||
c.traceResponse(tracedReq, res)
|
||||
c.traceResponse(req, tracedReq, res)
|
||||
|
||||
if res.StatusCode != 307 {
|
||||
return res, err
|
||||
|
@ -53,13 +53,13 @@ func (r *tracedRequest) Read(b []byte) (int, error) {
|
||||
return n, err
|
||||
}
|
||||
|
||||
func (c *Client) traceResponse(tracedReq *tracedRequest, res *http.Response) {
|
||||
func (c *Client) traceResponse(req *http.Request, tracedReq *tracedRequest, res *http.Response) {
|
||||
if tracedReq != nil {
|
||||
c.httpLogger.LogRequest(res.Request, tracedReq.BodySize)
|
||||
c.httpLogger.LogRequest(req, tracedReq.BodySize)
|
||||
}
|
||||
|
||||
if res == nil {
|
||||
c.httpLogger.LogResponse(res.Request, -1, 0)
|
||||
c.httpLogger.LogResponse(req, -1, 0)
|
||||
return
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user