lint

Author: Mzack9999

buggyhttp/buggyhttp.go

@@ -67,7 +67,7 @@ func messyEncoding(w http.ResponseWriter, req *http.Request) {
 	}
 
 	for _, oneencodingfrommany := range soManyEncodings {
-		fmt.Fprint(w, oneencodingfrommany)
+		_, _ = fmt.Fprint(w, oneencodingfrommany)
 	}
 }
 
@@ -77,7 +77,7 @@ func superSlow(w http.ResponseWriter, req *http.Request) {
 	z := w.(http.Flusher)
 	for name, headers := range req.Header {
 		for _, h := range headers {
-			fmt.Fprintf(w, "%v: %v\n", name, h)
+			_, _ = fmt.Fprintf(w, "%v: %v\n", name, h)
 			z.Flush()
 			time.Sleep(250 * time.Millisecond)
 		}
@@ -98,7 +98,9 @@ func superSlow(w http.ResponseWriter, req *http.Request) {
 func emptyResponse(w http.ResponseWriter, req *http.Request) {
 	hj, _ := w.(http.Hijacker)
 	conn, _, _ := hj.Hijack()
-	defer conn.Close()
+	defer func() {
+		_ = conn.Close()
+	}()
 }
 
 // SO MANY REDIRECTS
@@ -111,7 +113,9 @@ func infiniteRedirects(w http.ResponseWriter, req *http.Request) {
 func unexpectedEOF(w http.ResponseWriter, req *http.Request) {
 	hj, _ := w.(http.Hijacker)
 	conn, bufrw, _ := hj.Hijack()
-	defer conn.Close()
+	defer func() {
+		_ = conn.Close()
+	}()
 	// reply with bogus data - this should either crash the client or trigger a recoverable error on
 	// default retryablehttp requests
 	_, _ = bufrw.WriteString("HTTP/1.1 200 OK\n" +
@@ -121,18 +125,18 @@ func unexpectedEOF(w http.ResponseWriter, req *http.Request) {
 	// "Content-Length: -124" +
 	// "Content-Type: whatzdacontenttype" +
 	// "Connection: drunk")
-	bufrw.Flush()
+	_ = bufrw.Flush()
 }
 
 // Simulate normal 200 answer with body
 func foo(w http.ResponseWriter, req *http.Request) {
-	fmt.Fprintf(w, "foo")
+	_, _ = fmt.Fprintf(w, "foo")
 }
 
 // generates recoverable errors until SuccessAfter attempts => after it 200 + body
 var count int // as of now a local horrible variable suffice
 func successAfter(w http.ResponseWriter, req *http.Request) {
-	var successAfter int = defaultSuccessAfterThreshold
+	successAfter := defaultSuccessAfterThreshold
 	if req.FormValue("successAfter") != "" {
 		if i, err := strconv.Atoi(req.FormValue("successAfter")); err == nil {
 			successAfter = i
@@ -143,7 +147,9 @@ func successAfter(w http.ResponseWriter, req *http.Request) {
 	if count <= successAfter {
 		hj, _ := w.(http.Hijacker)
 		conn, bufrw, _ := hj.Hijack()
-		defer conn.Close()
+		defer func() {
+			_ = conn.Close()
+		}()
 		// reply with bogus data - this should either crash the client or trigger a recoverable error on
 		// default retryablehttp requests
 		_, _ = bufrw.WriteString("HHHTTP\\1,.1 -500 MAYBEOK\n" +
@@ -153,17 +159,15 @@ func successAfter(w http.ResponseWriter, req *http.Request) {
 			"Content-Length: -124\n" +
 			"Content-Type: whatzdacontenttype\n" +
 			"Connection: drunk")
-		bufrw.Flush()
+		_ = bufrw.Flush()
 		return
 	}
 
 	// zeroes attempts and return 200 + valid body
 	count = 0
-	fmt.Fprintf(w, "foo")
+	_, _ = fmt.Fprintf(w, "foo")
 }
 
-
-
 var (
 	server    *http.Server
 	serverTLS *http.Server

examples/main.go

@@ -45,7 +45,7 @@ func main() {
 
 	router := httprouter.New()
 	router.GET("/", func(w http.ResponseWriter, r *http.Request, _ httprouter.Params) {
-		fmt.Fprintf(w, "this is a test")
+		_, _ = fmt.Fprintf(w, "this is a test")
 	})
 	ts := httptest.NewServer(router)
 	defer ts.Close()
@@ -63,8 +63,8 @@ func main() {
 					log.Println(err)
 					continue
 				}
-				io.Copy(io.Discard, resp.Body)
-				resp.Body.Close()
+				_, _ = io.Copy(io.Discard, resp.Body)
+				_ = resp.Body.Close()
 			}
 		}()
 	}

request.go

@@ -159,7 +159,9 @@ func (r *Request) SetBodyString(body string) error {
 // read into the reusable reader.
 func (r *Request) SetBodyStream(bodyStream io.Reader, bodySize int64) error {
 	if closer, ok := bodyStream.(io.Closer); ok {
-		defer closer.Close()
+		defer func() {
+			_ = closer.Close()
+		}()
 
 		// Wrap in NopCloser to prevent NewReusableReadCloser from closing it,
 		// since we are handling the close via defer.

request_test.go

@@ -376,7 +376,7 @@ func verifyGetBody(t *testing.T, req *retryablehttp.Request, expected []byte) {
 		t.Fatalf("GetBody failed: %v", err)
 	}
 	data1, _ := io.ReadAll(rc1)
-	rc1.Close()
+	_ = rc1.Close()
 
 	if !bytes.Equal(data1, expected) {
 		t.Errorf("Read 1 mismatch. Got %s, want %s", string(data1), string(expected))
@@ -388,7 +388,7 @@ func verifyGetBody(t *testing.T, req *retryablehttp.Request, expected []byte) {
 		t.Fatalf("GetBody failed 2nd time: %v", err)
 	}
 	data2, _ := io.ReadAll(rc2)
-	rc2.Close()
+	_ = rc2.Close()
 
 	if !bytes.Equal(data2, expected) {
 		t.Errorf("Read 2 mismatch. Got %s, want %s", string(data2), string(expected))