#18 - returns response too when redirect error

client.go

@@ -571,21 +571,22 @@ func (c *Client) execute(req *Request) (*Response, error) {
 	req.Time = time.Now()
 	c.httpClient.Transport = c.transport
 	resp, err := c.httpClient.Do(req.RawRequest)
-	if err != nil {
-		return nil, err
-	}
 
 	response := &Response{
 		Request:     req,
 		RawResponse: resp,
 		receivedAt:  time.Now(),
 	}
 
+	if err != nil {
+		return response, err
+	}
+
 	if !req.isSaveResponse {
 		defer resp.Body.Close()
 		response.body, err = ioutil.ReadAll(resp.Body)
 		if err != nil {
-			return nil, err
+			return response, err
 		}
 
 		response.size = int64(len(response.body))

resty_test.go

@@ -857,7 +857,7 @@ func TestClientProxy(t *testing.T) {
 	c.SetProxy("http://sampleproxy:8888")
 
 	resp, err := c.R().Get(ts.URL)
-	assertEqual(t, true, resp == nil)
+	assertEqual(t, true, resp != nil)
 	assertEqual(t, true, err != nil)
 }
 
@@ -871,7 +871,7 @@ func TestPerRequestProxy(t *testing.T) {
 	resp, err := c.R().
 		SetProxy("http://sampleproxy:8888").
 		Get(ts.URL)
-	assertEqual(t, true, resp == nil)
+	assertEqual(t, true, resp != nil)
 	assertEqual(t, true, err != nil)
 }
 
@@ -886,7 +886,7 @@ func TestClientProxyOverride(t *testing.T) {
 	resp, err := c.R().
 		SetProxy("http://requestproxy:8888").
 		Get(ts.URL)
-	assertEqual(t, true, resp == nil)
+	assertEqual(t, true, resp != nil)
 	assertEqual(t, true, err != nil)
 }