fix: set max for max cookie number to math.MaxInt (#16388)

* fix: set max for max cookie number to `math.MaxInt32` Signed-off-by: OneMatchFox <[email protected]> * refactor: set max for max cookie number to `math.MaxInt` Co-authored-by: gdsoumya <[email protected]> Signed-off-by: Brian Fox <[email protected]> --------- Signed-off-by: OneMatchFox <[email protected]> Signed-off-by: Brian Fox <[email protected]> Co-authored-by: gdsoumya <[email protected]>
argoproj · Nov 20, 2023 · 61abc80 · 61abc80
1 parent 841339d
commit 61abc80
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/util/http/http.go b/util/http/http.go
@@ -30,7 +30,7 @@ const (
 
 // max number of chunks a cookie can be broken into. To be compatible with
 // widest range of browsers, you shouldn't create more than 30 cookies per domain
-var maxCookieNumber = env.ParseNumFromEnv(common.EnvMaxCookieNumber, 20, 0, math.MaxInt64)
+var maxCookieNumber = env.ParseNumFromEnv(common.EnvMaxCookieNumber, 20, 0, math.MaxInt)
 
 // MakeCookieMetadata generates a string representing a Web cookie.  Yum!
 func MakeCookieMetadata(key, value string, flags ...string) ([]string, error) {