Browse Source

chore(influx): no point of reading response when bad status (#16212)

There's no point in reading the body response without using
the result so removing it.

Fixes #9199
Ref #16207
Marcus Efraimsson 6 years ago
parent
commit
6b8e3122f8
1 changed files with 0 additions and 2 deletions
  1. 0 2
      pkg/tsdb/influxdb/influxdb.go

+ 0 - 2
pkg/tsdb/influxdb/influxdb.go

@@ -4,7 +4,6 @@ import (
 	"context"
 	"encoding/json"
 	"fmt"
-	"io/ioutil"
 	"net/http"
 	"net/url"
 	"path"
@@ -73,7 +72,6 @@ func (e *InfluxDBExecutor) Query(ctx context.Context, dsInfo *models.DataSource,
 
 	defer resp.Body.Close()
 	if resp.StatusCode/100 != 2 {
-		ioutil.ReadAll(resp.Body)
 		return nil, fmt.Errorf("Influxdb returned statuscode invalid status code: %v", resp.Status)
 	}