Code review suggestions applied

retry.go

@@ -6,7 +6,6 @@ import (
 	"bytes"
 	"context"
 	"fmt"
-	"golang.org/x/exp/slices"
 	"io"
 	"math/rand"
 	"net/http"
@@ -21,7 +20,6 @@ var random *rand.Rand
 
 var endpointsEligibleForRetry = []string{
 	loginRequestPath,
-	queryRequestPath,
 	tokenRequestPath,
 	authenticatorRequestPath,
 }
@@ -365,7 +363,7 @@ func (r *retryHTTP) isRetryableError(req *http.Request, res *http.Response, err
 	if res == nil {
 		return false, err
 	}
-	isRetryableURL := slices.Contains(endpointsEligibleForRetry, req.URL.Path)
-	isRetryableStatus := slices.Contains(statusCodesEligibleForRetry, res.StatusCode)
+	isRetryableURL := contains(endpointsEligibleForRetry, req.URL.Path)
+	isRetryableStatus := contains(statusCodesEligibleForRetry, res.StatusCode)
 	return isRetryableURL && isRetryableStatus, err
 }

util.go

@@ -250,3 +250,12 @@ type unixTimeProvider struct {
 func (utp *unixTimeProvider) currentTime() int64 {
 	return time.Now().UnixMilli()
 }
+
+func contains[T comparable](s []T, e T) bool {
+	for _, v := range s {
+		if v == e {
+			return true
+		}
+	}
+	return false
+}

util_test.go

@@ -364,3 +364,23 @@ func TestGetFromEnvFailOnMissing(t *testing.T) {
 		t.Error("should report error when there is missing env parameter")
 	}
 }
+
+type tcContains[T comparable] struct {
+	arr      []T
+	e        T
+	expected bool
+}
+
+func TestContains(t *testing.T) {
+	performContainsTestcase(tcContains[int]{[]int{1, 2, 3, 5}, 4, false}, t)
+	performContainsTestcase(tcContains[string]{[]string{"a", "b", "C", "F"}, "C", true}, t)
+	performContainsTestcase(tcContains[int]{[]int{1, 2, 3, 5}, 2, true}, t)
+	performContainsTestcase(tcContains[string]{[]string{"a", "b", "C", "F"}, "f", false}, t)
+}
+
+func performContainsTestcase[S comparable](tc tcContains[S], t *testing.T) {
+	result := contains(tc.arr, tc.e)
+	if result != tc.expected {
+		t.Errorf("contains failed; arr: %v, e: %v, should be %v but was %v", tc.arr, tc.e, tc.expected, result)
+	}
+}