🧹 replace ioutil with io package (#26)

client.go

@@ -4,7 +4,7 @@ import (
 	"bytes"
 	"context"
 	"encoding/json"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"strconv"
 
@@ -106,7 +106,7 @@ func (c *Client) DoClientRequest(ctx context.Context, client HTTPClient, url str
 			log.Debug().Str("body", spb.Message).Int("status", resp.StatusCode).Msg("non-ok http request")
 			return status.FromProto(spb).Err()
 		} else {
-			payload, err := ioutil.ReadAll(reader)
+			payload, err := io.ReadAll(reader)
 			if err != nil {
 				log.Error().Err(err).Msg("could not parse http body")
 			}
@@ -118,7 +118,7 @@ func (c *Client) DoClientRequest(ctx context.Context, client HTTPClient, url str
 		return errors.Wrap(err, "aborted because context was done")
 	}
 
-	respBodyBytes, err := ioutil.ReadAll(resp.Body)
+	respBodyBytes, err := io.ReadAll(resp.Body)
 	if err != nil {
 		return errors.Wrap(err, "failed to read response body")
 	}

error.go

@@ -2,7 +2,6 @@ package ranger
 
 import (
 	"io"
-	"io/ioutil"
 	"net/http"
 
 	"github.com/rs/zerolog/log"
@@ -51,7 +50,7 @@ func HttpError(w http.ResponseWriter, req *http.Request, err error) {
 
 // parseStatus tries to parse the proto Status from the body of the response.
 func parseStatus(reader io.Reader) (*spb.Status, error) {
-	payload, err := ioutil.ReadAll(reader)
+	payload, err := io.ReadAll(reader)
 	if err != nil {
 		return nil, err
 	}

examples/oneof/oneof_test.go

@@ -3,7 +3,7 @@ package oneof
 import (
 	"bytes"
 	"context"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"net/http/httptest"
 	"testing"
@@ -82,7 +82,7 @@ func TestOneOfJSON(t *testing.T) {
 
 		// check response
 		r := &OneOfReply{}
-		data, err := ioutil.ReadAll(resp.Body)
+		data, err := io.ReadAll(resp.Body)
 		require.NoError(t, err)
 		err = jsonpb.Unmarshal(data, r)
 		assert.Nil(t, err)
@@ -106,7 +106,7 @@ func TestOneOfJSON(t *testing.T) {
 
 		// check response
 		r := &OneOfReply{}
-		data, err := ioutil.ReadAll(resp.Body)
+		data, err := io.ReadAll(resp.Body)
 		require.NoError(t, err)
 		err = jsonpb.Unmarshal(data, r)
 		assert.Nil(t, err)

server.go

@@ -3,7 +3,7 @@ package ranger
 import (
 	"context"
 	"fmt"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"strings"
 
@@ -104,7 +104,7 @@ func (s *server) ServeHTTP(w http.ResponseWriter, req *http.Request) {
 // the context. It also adds the http headers to the context.
 func preProcessRequest(ctx context.Context, req *http.Request) (context.Context, context.CancelFunc, []byte, error) {
 	// read body content
-	body, err := ioutil.ReadAll(req.Body)
+	body, err := io.ReadAll(req.Body)
 	defer req.Body.Close()
 	if err != nil {
 		return nil, nil, nil, status.Error(codes.DataLoss, "unrecoverable data loss or corruption")

server_test.go

@@ -5,7 +5,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"io"
-	"io/ioutil"
 	"net/http"
 	"net/http/httptest"
 	"testing"
@@ -62,7 +61,6 @@ func TestRangerHttpServer(t *testing.T) {
 	srv.ServeHTTP(w, req)
 	resp = w.Result()
 	assert.Equal(t, 200, resp.StatusCode, "correct status code")
-
 }
 
 func TestRangerHttpHeader(t *testing.T) {
@@ -97,7 +95,7 @@ func TestRangerHttpHeader(t *testing.T) {
 	resp = w.Result()
 	assert.Equal(t, 200, resp.StatusCode, "correct status code")
 
-	content, err := ioutil.ReadAll(w.Body)
+	content, err := io.ReadAll(w.Body)
 	assert.Nil(t, err, "should return protobuf content")
 
 	var msg pingpong.PongReply
@@ -117,7 +115,7 @@ func runStatusCall(srv http.Handler, path string) *http.Response {
 }
 
 func parseStatus(reader io.Reader) (*spb.Status, error) {
-	payload, err := ioutil.ReadAll(reader)
+	payload, err := io.ReadAll(reader)
 	if err != nil {
 		return nil, err
 	}
@@ -163,7 +161,7 @@ func TestRangerErrorHandling(t *testing.T) {
 	srv.ServeHTTP(w, req)
 	resp = w.Result()
 	assert.Equal(t, 400, resp.StatusCode, "correct status code")
-	payload, err := ioutil.ReadAll(resp.Body)
+	payload, err := io.ReadAll(resp.Body)
 	if err != nil {
 		t.Fatal(err)
 	}