opt: use net/url for validating network addresses (#728)

Author: panjf2000

.github/workflows/cross-compile-bsd.yml

@@ -31,7 +31,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        go: ['1.20', '1.24']
+        go: ['1.20', '1.25']
         os:
           - ubuntu-latest
     name: Go ${{ matrix.go }} @ ${{ matrix.os }}

.github/workflows/test.yml

@@ -56,10 +56,10 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        go: ['1.20', '1.24']
+        go: ['1.20', '1.25']
         os:
           - ubuntu-latest
-          - macos-latest
+          - macos-14 # check out this issue: https://github.com/actions/runner-images/issues/10924
           - windows-latest
     name: Go ${{ matrix.go }} @ ${{ matrix.os }}
     runs-on: ${{ matrix.os }}

.github/workflows/test_gc_opt.yml

@@ -56,10 +56,10 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        go: ['1.20', '1.24']
+        go: ['1.20', '1.25']
         os:
           - ubuntu-latest
-          - macos-latest
+          - macos-14 # check out this issue: https://github.com/actions/runner-images/issues/10924
           - windows-latest
     name: Go ${{ matrix.go }} @ ${{ matrix.os }}
     runs-on: ${{ matrix.os }}

.github/workflows/test_poll_opt.yml

@@ -55,8 +55,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        go: ['1.20', '1.24']
-        os: [ubuntu-latest, macos-latest]
+        go: ['1.20', '1.25']
+        os: [ubuntu-latest, macos-14] # check out this issue: https://github.com/actions/runner-images/issues/10924
     name: Go ${{ matrix.go }} @ ${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     steps:

.github/workflows/test_poll_opt_gc_opt.yml

@@ -55,8 +55,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        go: ['1.20', '1.24']
-        os: [ubuntu-latest, macos-latest]
+        go: ['1.20', '1.25']
+        os: [ubuntu-latest, macos-14] # check out this issue: https://github.com/actions/runner-images/issues/10924
     name: Go ${{ matrix.go }} @ ${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     steps:

gnet.go

@@ -22,6 +22,7 @@ import (
 	"context"
 	"io"
 	"net"
+	"net/url"
 	"runtime"
 	"strings"
 	"sync"
@@ -746,21 +747,25 @@ func Stop(ctx context.Context, protoAddr string) error {
 }
 
 func parseProtoAddr(protoAddr string) (string, string, error) {
-	protoAddr = strings.ToLower(protoAddr)
-	if strings.Count(protoAddr, "://") != 1 {
+	// Percent-encode "%" in the address to avoid url.Parse error.
+	// For example: udp://[ff02::3%lo0]:9991
+	protoAddr = strings.ReplaceAll(protoAddr, "%", "%25")
+
+	u, err := url.Parse(protoAddr)
+	if err != nil {
+		return "", "", err
+	}
+
+	if u.Scheme == "" || u.Host == "" || u.Path != "" {
 		return "", "", errorx.ErrInvalidNetworkAddress
 	}
-	pair := strings.SplitN(protoAddr, "://", 2)
-	proto, addr := pair[0], pair[1]
-	switch proto {
+	switch u.Scheme {
 	case "tcp", "tcp4", "tcp6", "udp", "udp4", "udp6", "unix":
 	default:
 		return "", "", errorx.ErrUnsupportedProtocol
 	}
-	if addr == "" {
-		return "", "", errorx.ErrInvalidNetworkAddress
-	}
-	return proto, addr, nil
+
+	return u.Scheme, u.Host, nil
 }
 
 func determineEventLoops(opts *Options) int {