From 896114d33030936b193ae79364880aea1e5855da Mon Sep 17 00:00:00 2001 From: Chris Lu Date: Sat, 4 Apr 2026 11:50:43 -0700 Subject: [PATCH] fix(admin): fix master leader link showing incorrect port in Admin UI (#8924) fix(admin): use gRPC address for current server in RaftListClusterServers The old Raft implementation was returning the HTTP address (ms.option.Master) for the current server, while peers used gRPC addresses (peer.ConnectionString). The Admin UI's GetClusterMasters() converts all addresses from gRPC to HTTP via GrpcAddressToServerAddress (port - 10000), which produced a negative port (-667) for the current server since its address was already in HTTP format (port 9333). Use ToGrpcAddress() for consistency with both HashicorpRaft (which stores gRPC addresses) and old Raft peers. Fixes #8921 --- weed/server/master_grpc_server_raft.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/weed/server/master_grpc_server_raft.go b/weed/server/master_grpc_server_raft.go index aa7c6ff28..1c0abea51 100644 --- a/weed/server/master_grpc_server_raft.go +++ b/weed/server/master_grpc_server_raft.go @@ -41,7 +41,7 @@ func (ms *MasterServer) RaftListClusterServers(ctx context.Context, req *master_ // Add the current server itself (Peers() only returns other peers) resp.ClusterServers = append(resp.ClusterServers, &master_pb.RaftListClusterServersResponse_ClusterServers{ Id: currentServerName, - Address: string(ms.option.Master), + Address: ms.option.Master.ToGrpcAddress(), Suffrage: "Voter", IsLeader: currentServerName == leader, })