Merge pull request #152 from thaJeztah/bump_go_version

update minimum go version to go1.23

Author: thaJeztah

go.mod

@@ -1,6 +1,6 @@
 module github.com/docker/go-connections
 
-go 1.18
+go 1.23
 
 require (
 	github.com/Microsoft/go-winio v0.4.21

nat/nat.go

@@ -218,7 +218,7 @@ func ParsePortSpec(rawPort string) ([]PortMapping, error) {
 	count := endPort - startPort + 1
 	ports := make([]PortMapping, 0, count)
 
-	for i := 0; i < count; i++ {
+	for i := range count {
 		hPort := ""
 		if hostPort != "" {
 			hPort = strconv.Itoa(startHostPort + i)

nat/sort.go

@@ -69,7 +69,6 @@ func SortPortMap(ports []Port, bindings map[Port][]PortBinding) {
 		portInt, portProto := p.Int(), p.Proto()
 		if binding, ok := bindings[p]; ok && len(binding) > 0 {
 			for _, b := range binding {
-				b := b // capture loop variable for go < 1.22
 				s = append(s, portMapEntry{
 					port: p, binding: &b,
 					portInt: portInt, portProto: portProto,

nat/sort_test.go

@@ -83,7 +83,7 @@ func BenchmarkSortPortMap(b *testing.B) {
 	ports := make([]Port, 0, n*2)
 	portMap := make(map[Port][]PortBinding, n*2)
 
-	for i := 0; i < n; i++ {
+	for i := range n {
 		portNum := 30000 + (i % 50) // force duplicate port numbers
 		tcp := Port(fmt.Sprintf("%d/tcp", portNum))
 		udp := Port(fmt.Sprintf("%d/udp", portNum))

proxy/logger.go

@@ -1,11 +1,11 @@
 package proxy
 
 type logger interface {
-	Printf(format string, args ...interface{})
+	Printf(format string, args ...any)
 }
 
 type noopLogger struct{}
 
-func (l *noopLogger) Printf(_ string, _ ...interface{}) {
+func (l *noopLogger) Printf(_ string, _ ...any) {
 	// Do nothing :)
 }

tlsconfig/config_test.go

@@ -77,7 +77,7 @@ func TestConfigServerTLSFailsIfUnableToLoadCerts(t *testing.T) {
 	_ = tempFile.Close()
 
 	for _, badFile := range []string{"not-a-file", tempFile.Name()} {
-		for i := 0; i < 3; i++ {
+		for i := range 3 {
 			files := []string{cert, key, ca}
 			files[i] = badFile
 
@@ -475,7 +475,7 @@ func TestConfigClientTLSClientCertOrKeyInvalid(t *testing.T) {
 	defer func() { _ = os.Remove(tempFile.Name()) }()
 	_ = tempFile.Close()
 
-	for i := 0; i < 2; i++ {
+	for i := range 2 {
 		for _, invalid := range []string{"not-a-file", "", tempFile.Name()} {
 			files := []string{cert, key}
 			files[i] = invalid