chore: enable errorlint

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>

Author: mmorel-35

.golangci.yml

@@ -5,6 +5,9 @@ run:
   timeout: 5m
 
 linters:
+  enable:
+    - errcheck
+    - errorlint
   settings:
     errcheck:
       exclude-functions:
@@ -23,3 +26,9 @@ formatters:
   settings:
     gofumpt:
       extra-rules: true
+
+issues:
+  # Maximum issues count per one linter.
+  max-issues-per-linter: 0
+  # Maximum count of issues with the same text.
+  max-same-issues: 0

interceptors/client.go

@@ -7,6 +7,7 @@ package interceptors
 
 import (
 	"context"
+	"errors"
 	"io"
 	"time"
 
@@ -68,7 +69,7 @@ func (s *monitoredClientStream) RecvMsg(m any) error {
 			return nil
 		}
 		var postErr error
-		if err != io.EOF {
+		if !errors.Is(err, io.EOF) {
 			postErr = err
 		}
 		s.reporter.PostCall(postErr, time.Since(s.startTime))

interceptors/client_test.go

@@ -5,6 +5,7 @@ package interceptors
 
 import (
 	"context"
+	"errors"
 	"io"
 	"net"
 	"sync"
@@ -249,7 +250,7 @@ func (s *ClientInterceptorTestSuite) TestListReporting() {
 	count := 0
 	for {
 		_, err := ss.Recv()
-		if err == io.EOF {
+		if errors.Is(err, io.EOF) {
 			break
 		}
 		require.NoError(s.T(), err, "reading pingList shouldn't fail")
@@ -305,7 +306,7 @@ func (s *ClientInterceptorTestSuite) TestBiStreamingReporting() {
 		for s.ctx.Err() == nil {
 
 			_, err := ss.Recv()
-			if err == io.EOF {
+			if errors.Is(err, io.EOF) {
 				break
 			}
 			if !s.Assert().NoError(err, "reading pingStream shouldn't fail") {

interceptors/logging/interceptors.go

@@ -5,6 +5,7 @@ package logging
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"io"
 	"time"
@@ -31,7 +32,7 @@ func (c *reporter) PostCall(err error, duration time.Duration) {
 	if !has(c.opts.loggableEvents, FinishCall) {
 		return
 	}
-	if err == io.EOF {
+	if errors.Is(err, io.EOF) {
 		err = nil
 	}
 

interceptors/logging/interceptors_test.go

@@ -7,6 +7,7 @@ import (
 	"bytes"
 	"context"
 	"encoding/json"
+	"errors"
 	"fmt"
 	"io"
 	"runtime"
@@ -283,7 +284,7 @@ func (s *loggingClientServerSuite) TestPingList() {
 	require.NoError(s.T(), err, "should not fail on establishing the stream")
 	for {
 		_, err := stream.Recv()
-		if err == io.EOF {
+		if errors.Is(err, io.EOF) {
 			break
 		}
 		require.NoError(s.T(), err, "reading stream should not fail")
@@ -468,7 +469,7 @@ func (s *loggingCustomDurationSuite) TestPingList_HasOverriddenDuration() {
 	require.NoError(s.T(), err, "should not fail on establishing the stream")
 	for {
 		_, err := stream.Recv()
-		if err == io.EOF {
+		if errors.Is(err, io.EOF) {
 			break
 		}
 		require.NoError(s.T(), err, "reading stream should not fail")
@@ -754,7 +755,7 @@ func (s *loggingCustomGrpcLogFieldsSuite) TestCustomGrpcLogFieldsWithPingList()
 	require.NoError(s.T(), err, "should not fail on establishing the stream")
 	for {
 		_, err := stream.Recv()
-		if err == io.EOF {
+		if errors.Is(err, io.EOF) {
 			break
 		}
 		require.NoError(s.T(), err, "reading stream should not fail")

interceptors/retry/retry.go

@@ -5,6 +5,7 @@ package retry
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"io"
 	"sync"
@@ -213,7 +214,7 @@ func (s *serverStreamingRetryingStream) RecvMsg(m any) error {
 
 func (s *serverStreamingRetryingStream) receiveMsgAndIndicateRetry(m any) (bool, error) {
 	err := s.getStream().RecvMsg(m)
-	if err == nil || err == io.EOF {
+	if err == nil || errors.Is(err, io.EOF) {
 		return false, err
 	}
 	if isContextError(err) {
@@ -308,10 +309,10 @@ func perStreamContext(parentCtx context.Context, callOpts *options, attempt uint
 }
 
 func contextErrToGrpcErr(err error) error {
-	switch err {
-	case context.DeadlineExceeded:
+	switch {
+	case errors.Is(err, context.DeadlineExceeded):
 		return status.Error(codes.DeadlineExceeded, err.Error())
-	case context.Canceled:
+	case errors.Is(err, context.Canceled):
 		return status.Error(codes.Canceled, err.Error())
 	default:
 		return status.Error(codes.Unknown, err.Error())

interceptors/server_test.go

@@ -5,6 +5,7 @@ package interceptors
 
 import (
 	"context"
+	"errors"
 	"io"
 	"net"
 	"sync"
@@ -111,7 +112,7 @@ func (s *ServerInterceptorTestSuite) TestStreamingReports() {
 	count := 0
 	for {
 		_, err := ss.Recv()
-		if err == io.EOF {
+		if errors.Is(err, io.EOF) {
 			break
 		}
 		require.NoError(s.T(), err, "reading pingList shouldn't fail")
@@ -153,7 +154,7 @@ func (s *ServerInterceptorTestSuite) TestBiStreamingReporting() {
 		for s.ctx.Err() == nil {
 
 			_, err := ss.Recv()
-			if err == io.EOF {
+			if errors.Is(err, io.EOF) {
 				break
 			}
 			require.NoError(s.T(), err, "reading pingStream shouldn't fail")

providers/prometheus/client_test.go

@@ -4,6 +4,7 @@
 package prometheus
 
 import (
+	"errors"
 	"io"
 	"testing"
 
@@ -74,7 +75,7 @@ func (s *ClientInterceptorTestSuite) TestStreamingIncrementsMetrics() {
 	count := 0
 	for {
 		_, err = ss.Recv()
-		if err == io.EOF {
+		if errors.Is(err, io.EOF) {
 			break
 		}
 		require.NoError(s.T(), err, "reading pingList shouldn't fail")

providers/prometheus/server_test.go

@@ -6,6 +6,7 @@ package prometheus
 import (
 	"bufio"
 	"context"
+	"errors"
 	"fmt"
 	"io"
 	"net/http"
@@ -127,7 +128,7 @@ func (s *ServerInterceptorTestSuite) TestStreamingIncrementsMetrics() {
 	count := 0
 	for {
 		_, err := ss.Recv()
-		if err == io.EOF {
+		if errors.Is(err, io.EOF) {
 			break
 		}
 		require.NoError(s.T(), err, "reading pingList shouldn't fail")
@@ -238,7 +239,7 @@ func toFloat64HistCount(h prometheus.Observer) uint64 {
 
 	pb := &dto.Metric{}
 	if err := m.Write(pb); err != nil {
-		panic(fmt.Errorf("metric write failed, err=%v", err))
+		panic(fmt.Errorf("metric write failed: %w", err))
 	}
 
 	if pb.Histogram != nil {