chore: better addr parsing

Author: wwqgtxx

listener/sing_shadowsocks/server.go

@@ -196,7 +196,7 @@ func (l *Listener) HandleConn(conn net.Conn, tunnel C.Tunnel, additions ...inbou
 	ctx := sing.WithAdditions(context.TODO(), additions...)
 	err := l.service.NewConnection(ctx, conn, M.Metadata{
 		Protocol: "shadowsocks",
-		Source:   M.ParseSocksaddr(conn.RemoteAddr().String()),
+		Source:   M.SocksaddrFromNet(conn.RemoteAddr()),
 	})
 	if err != nil {
 		_ = conn.Close()

listener/sing_vless/server.go

@@ -201,7 +201,7 @@ func (l *Listener) HandleConn(conn net.Conn, tunnel C.Tunnel, additions ...inbou
 	ctx := sing.WithAdditions(context.TODO(), additions...)
 	err := l.service.NewConnection(ctx, conn, metadata.Metadata{
 		Protocol: "vless",
-		Source:   metadata.ParseSocksaddr(conn.RemoteAddr().String()),
+		Source:   metadata.SocksaddrFromNet(conn.RemoteAddr()),
 	})
 	if err != nil {
 		_ = conn.Close()

listener/sing_vmess/server.go

@@ -187,7 +187,7 @@ func (l *Listener) HandleConn(conn net.Conn, tunnel C.Tunnel, additions ...inbou
 	ctx := sing.WithAdditions(context.TODO(), additions...)
 	err := l.service.NewConnection(ctx, conn, metadata.Metadata{
 		Protocol: "vmess",
-		Source:   metadata.ParseSocksaddr(conn.RemoteAddr().String()),
+		Source:   metadata.SocksaddrFromNet(conn.RemoteAddr()),
 	})
 	if err != nil {
 		_ = conn.Close()

transport/socks5/socks5.go

@@ -189,7 +189,7 @@ func ServerHandshake(rw net.Conn, authenticator auth.Authenticator) (addr Addr,
 	switch command {
 	case CmdConnect, CmdUDPAssociate:
 		// Acquire server listened address info
-		localAddr := ParseAddr(rw.LocalAddr().String())
+		localAddr := ParseAddrToSocksAddr(rw.LocalAddr())
 		if localAddr == nil {
 			err = ErrAddressNotSupported
 		} else {
@@ -414,12 +414,15 @@ func ParseAddr(s string) Addr {
 func ParseAddrToSocksAddr(addr net.Addr) Addr {
 	var hostip net.IP
 	var port int
-	if udpaddr, ok := addr.(*net.UDPAddr); ok {
-		hostip = udpaddr.IP
-		port = udpaddr.Port
-	} else if tcpaddr, ok := addr.(*net.TCPAddr); ok {
-		hostip = tcpaddr.IP
-		port = tcpaddr.Port
+	switch addr := addr.(type) {
+	case *net.UDPAddr:
+		hostip = addr.IP
+		port = addr.Port
+	case *net.TCPAddr:
+		hostip = addr.IP
+		port = addr.Port
+	case nil:
+		return nil
 	}
 
 	// fallback parse