From dd092f6c2bb68e830be001f8cf1472fc15e8ab4a Mon Sep 17 00:00:00 2001 From: Anis Elleuch Date: Mon, 26 Nov 2018 18:49:38 +0100 Subject: [PATCH] gateway: Properly set globalMinioPort (#6859) globalMinioPort is used in federation which stores the address and the port number of the server hosting the specified bucket, this latter uses globalMinioPort but this latter is not set in startup of the gateway mode. This commit fixes the behavior. --- cmd/gateway-main.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/cmd/gateway-main.go b/cmd/gateway-main.go index 5a55dfb88..03cc28456 100644 --- a/cmd/gateway-main.go +++ b/cmd/gateway-main.go @@ -135,7 +135,8 @@ func StartGateway(ctx *cli.Context, gw Gateway) { handleCommonCmdArgs(ctx) // Get port to listen on from gateway address - _, gatewayPort, pErr := net.SplitHostPort(gatewayAddr) + var pErr error + _, globalMinioPort, pErr = net.SplitHostPort(gatewayAddr) if pErr != nil { logger.FatalIf(pErr, "Unable to start gateway") } @@ -144,7 +145,7 @@ func StartGateway(ctx *cli.Context, gw Gateway) { // to IPv6 address ie minio will start listening on IPv6 address whereas another // (non-)minio process is listening on IPv4 of given port. // To avoid this error situation we check for port availability. - logger.FatalIf(checkPortAvailability(gatewayPort), "Unable to start the gateway") + logger.FatalIf(checkPortAvailability(globalMinioPort), "Unable to start the gateway") // Create certs path. logger.FatalIf(createConfigDir(), "Unable to create configuration directories")