From 03c65c96199c861f29ea71eb330f0fbe5dfe02f3 Mon Sep 17 00:00:00 2001
From: Aliaksandr Valialkin <valyala@victoriametrics.com>
Date: Wed, 6 Mar 2024 17:35:38 +0200
Subject: [PATCH] app/vmauth: use slices.Contains() instead of hasInt()
---
app/vmauth/main.go | 12 ++----------
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/app/vmauth/main.go b/app/vmauth/main.go
index d0327fc60b..4f24a9d5da 100644
--- a/app/vmauth/main.go
+++ b/app/vmauth/main.go
@@ -13,6 +13,7 @@ import (
"net/textproto"
"net/url"
"os"
+ "slices"
"strings"
"sync"
"time"
@@ -271,7 +272,7 @@ func tryProcessingRequest(w http.ResponseWriter, r *http.Request, targetURL *url
logger.Warnf("remoteAddr: %s; requestURI: %s; retrying the request to %s because of response error: %s", remoteAddr, req.URL, targetURL, err)
return false
}
- if hasInt(retryStatusCodes, res.StatusCode) {
+ if slices.Contains(retryStatusCodes, res.StatusCode) {
_ = res.Body.Close()
if !rtbOK || !rtb.canRetry() {
// If we get an error from the retry_status_codes list, but cannot execute retry,
@@ -313,15 +314,6 @@ func tryProcessingRequest(w http.ResponseWriter, r *http.Request, targetURL *url
return true
}
-func hasInt(a []int, n int) bool {
- for _, x := range a {
- if x == n {
- return true
- }
- }
- return false
-}
-
var copyBufPool bytesutil.ByteBufferPool
func copyHeader(dst, src http.Header) {