change maxport from maxInt to 65535

housepower · Apr 17, 2024 · d379713 · d379713
1 parent aca7950
commit d379713
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/cmd/clickhouse_sinker/main.go b/cmd/clickhouse_sinker/main.go
@@ -49,10 +49,6 @@ var (
 	runner      *task.Sinker
 )
 
-const (
-	HttpPortBase = 10000
-)
-
 func initCmdOptions() {
 	// 1. Set options to default value.
 	cmdOps = util.CmdOptions{
@@ -201,7 +197,7 @@ func main() {
 		// cmdOps.HTTPPort=0: let OS choose the listen port, and record the exact metrics URL to log.
 		httpPort := cmdOps.HTTPPort
 		if httpPort == 0 {
-			httpPort = HttpPortBase
+			httpPort = util.HttpPortBase
 		}
 		httpPort = util.GetSpareTCPPort(httpPort)
 

diff --git a/util/net.go b/util/net.go
@@ -17,12 +17,16 @@ package util
 
 import (
 	"fmt"
-	"math"
 	"net"
 
 	"github.com/thanos-io/thanos/pkg/errors"
 )
 
+const (
+	HttpPortBase = 10000
+	MaxPort      = 65535
+)
+
 func GetIP4Byname(host string) (ips []string, err error) {
 	addrs, err := net.LookupIP(host)
 	if err != nil {
@@ -53,7 +57,7 @@ func GetOutboundIP() (ip net.IP, err error) {
 
 // GetSpareTCPPort finds a spare TCP port.
 func GetSpareTCPPort(portBegin int) int {
-	for port := portBegin; port < math.MaxInt; port++ {
+	for port := portBegin; port <= MaxPort; port++ {
 		if err := testListenOnPort(port); err == nil {
 			return port
 		}