ioutil -> io

Closes #27

Author: mdwhatcott

internal/sdk/doc_test.go

@@ -1,7 +1,7 @@
 package sdk
 
 import (
-	"io/ioutil"
+	"io"
 	"net/http"
 )
 
@@ -35,7 +35,7 @@ func (f *FakeMultiHTTPClient) Do(request *http.Request) (*http.Response, error)
 
 func (f *FakeMultiHTTPClient) simulateServerReadingRequestBody(request *http.Request) {
 	if request.Body != nil {
-		body, _ := ioutil.ReadAll(request.Body)
+		body, _ := io.ReadAll(request.Body)
 		f.bodies = append(f.bodies, string(body))
 	} else {
 		f.bodies = append(f.bodies, request.URL.Query().Get("body"))

internal/sdk/http_sender.go

@@ -1,7 +1,7 @@
 package sdk
 
 import (
-	"io/ioutil"
+	"io"
 	"net/http"
 
 	"github.com/smartystreets/smartystreets-go-sdk"
@@ -38,7 +38,7 @@ func readResponseBody(response *http.Response) ([]byte, error) {
 	// TODO: Since we already copy response.Body in retry_client.go -> readBody()
 	//       It would behoove us to prevent a second copy in that case.
 
-	if content, err := ioutil.ReadAll(response.Body); err != nil {
+	if content, err := io.ReadAll(response.Body); err != nil {
 		_ = response.Body.Close()
 		return nil, err
 	} else {

internal/sdk/keep_alive_close_client_test.go

@@ -1,7 +1,7 @@
 package sdk
 
 import (
-	"io/ioutil"
+	"io"
 	"net/http"
 	"net/http/httptest"
 	"strings"
@@ -26,7 +26,7 @@ func (f *KeepAliveCloseClientFixture) Setup() {
 	f.inner.response = &http.Response{
 		ProtoMajor: 1, ProtoMinor: 1,
 		StatusCode: http.StatusTeapot,
-		Body:       ioutil.NopCloser(strings.NewReader("Goodbye, World!")),
+		Body:       io.NopCloser(strings.NewReader("Goodbye, World!")),
 	}
 	f.request = httptest.NewRequest("GET", "/", nil)
 }

internal/sdk/retry_client.go

@@ -3,7 +3,6 @@ package sdk
 import (
 	"bytes"
 	"io"
-	"io/ioutil"
 	"math/rand"
 	"net/http"
 	"sync"
@@ -54,13 +53,13 @@ func (r *RetryClient) doGet(request *http.Request) (response *http.Response, err
 }
 
 func (r *RetryClient) doBufferedPost(request *http.Request) (response *http.Response, err error) {
-	body, err := ioutil.ReadAll(request.Body)
+	body, err := io.ReadAll(request.Body)
 	if err != nil {
 		return nil, err
 	}
 
 	for attempt := 0; r.backOff(attempt); attempt++ {
-		request.Body = ioutil.NopCloser(bytes.NewReader(body))
+		request.Body = io.NopCloser(bytes.NewReader(body))
 		if response, err = r.inner.Do(request); err == nil && response.StatusCode == http.StatusOK {
 			if r.readBody(response) {
 				break

us-extract-api/lookup.go

@@ -1,11 +1,12 @@
 package extract
 
 import (
-	"github.com/smartystreets/smartystreets-go-sdk/us-street-api"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"strconv"
 	"strings"
+
+	"github.com/smartystreets/smartystreets-go-sdk/us-street-api"
 )
 
 // Lookup represents all input fields documented here:
@@ -64,7 +65,7 @@ func (l *Lookup) setBody(request *http.Request) {
 	}
 
 	body := strings.NewReader(l.Text)
-	request.Body = ioutil.NopCloser(body)
+	request.Body = io.NopCloser(body)
 	request.ContentLength = int64(body.Len())
 }
 func (l *Lookup) setHeaders(request *http.Request) {

us-extract-api/lookup_test.go

@@ -1,7 +1,7 @@
 package extract
 
 import (
-	"io/ioutil"
+	"io"
 	"net/http"
 	"net/url"
 	"testing"
@@ -33,7 +33,7 @@ func (f *LookupFixture) query() url.Values {
 }
 
 func readBody(request *http.Request) string {
-	bytes, _ := ioutil.ReadAll(request.Body)
+	bytes, _ := io.ReadAll(request.Body)
 	return string(bytes)
 }
 

us-street-api/batch.go

@@ -3,7 +3,7 @@ package street
 import (
 	"bytes"
 	"encoding/json"
-	"io/ioutil"
+	"io"
 	"net/http"
 )
 
@@ -79,7 +79,7 @@ func (b *Batch) serializeGET(request *http.Request) {
 func (b *Batch) serializePOST(request *http.Request) {
 	request.Method = http.MethodPost
 	payload, _ := json.Marshal(b.lookups) // We control the types being serialized. This is safe.
-	request.Body = ioutil.NopCloser(bytes.NewReader(payload))
+	request.Body = io.NopCloser(bytes.NewReader(payload))
 	request.ContentLength = int64(len(payload))
 	request.Header.Set("Content-Type", "application/json")
 }

us-street-api/batch_processing_test.go

@@ -3,7 +3,7 @@ package street
 import (
 	"errors"
 	"fmt"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"strconv"
 	"testing"
@@ -130,7 +130,7 @@ func (f *FakeMultiSender) Send(request *http.Request) ([]byte, error) {
 	f.requests = append(f.requests, request)
 
 	if request.Body != nil {
-		body, _ := ioutil.ReadAll(request.Body)
+		body, _ := io.ReadAll(request.Body)
 		f.requestBodies = append(f.requestBodies, string(body))
 	}
 

us-street-api/client_test.go

@@ -3,7 +3,7 @@ package street
 import (
 	"context"
 	"errors"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"net/url"
 	"testing"
@@ -292,7 +292,7 @@ func (f *FakeSender) Send(request *http.Request) ([]byte, error) {
 	f.callCount++
 	f.request = request
 	if request != nil && request.Body != nil {
-		f.requestBody, _ = ioutil.ReadAll(request.Body)
+		f.requestBody, _ = io.ReadAll(request.Body)
 	}
 	return []byte(f.response), f.err
 }

us-zipcode-api/batch.go

@@ -3,7 +3,7 @@ package zipcode
 import (
 	"bytes"
 	"encoding/json"
-	"io/ioutil"
+	"io"
 	"net/http"
 )
 
@@ -77,7 +77,7 @@ func (b *Batch) serializeGET(request *http.Request) {
 func (b *Batch) serializePOST(request *http.Request) {
 	request.Method = http.MethodPost
 	payload, _ := json.Marshal(b.lookups) // We control the types being serialized. This is safe.
-	request.Body = ioutil.NopCloser(bytes.NewReader(payload))
+	request.Body = io.NopCloser(bytes.NewReader(payload))
 	request.ContentLength = int64(len(payload))
 	request.Header.Set("Content-Type", "application/json")
 }