actually return error

only try to read response if there was no error
willmadison · Oct 31, 2012 · ba9c3b7 · ba9c3b7
1 parent ce231a5
commit ba9c3b7
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/api/request.go b/api/request.go
@@ -83,12 +83,13 @@ func (r *Request) SetBody(body io.Reader) {
 
 func (r *Request) Do(v interface{}) (string, error) {
 	res, err := http.DefaultClient.Do((*http.Request)(r))
-	defer res.Body.Close()
-	bodyBytes, err := ioutil.ReadAll(res.Body)
 	if err != nil {
-		return "", nil
+		return "", err
 	}
 
+	defer res.Body.Close()
+	bodyBytes, err := ioutil.ReadAll(res.Body)
+
 	if v != nil {
 		jsonErr := json.Unmarshal(bodyBytes, v)
 		if jsonErr != nil {