Regression, no longer host:port reported in show processlist

[mjonss]

Bug Report

Please answer these questions before submitting your issue. Thanks!

In #10903 the host:port was added in show processlist, but there has been a regression, resulting it it does no longer show up.

1. Minimal reproduce step (Required)

See #10903

2. What did you expect to see? (Required)

3. What did you see instead (Required)

4. What is your TiDB version? (Required)

Master branch 7a8c7f8

[mjonss]

Reason being cc.PeerHost() has been called before, but only returns "" for port on second call.

Fixed by something like:

diff --git a/server/conn.go b/server/conn.go
index f74c00a555..31ccba25cf 100644
--- a/server/conn.go
+++ b/server/conn.go
@@ -888,7 +888,7 @@ func (cc *clientConn) checkAuthPlugin(ctx context.Context, resp *handshakeRespon
 
 func (cc *clientConn) PeerHost(hasPassword string) (host, port string, err error) {
        if len(cc.peerHost) > 0 {
-               return cc.peerHost, "", nil
+               return cc.peerHost, cc.peerPort, nil
        }
        host = variable.DefHostname
        if cc.isUnixSocket {

[mjonss]

/assign

[github-actions]

Please check whether the issue should be labeled with 'affects-x.y' or 'fixes-x.y.z', and then remove 'needs-more-info' label.