Pick up server address from --address option (#3002) (#3008)

This makes sure that when SSL is enabled (for FS/single node mode),
the server address is picked up from the --address option (that needs
to include the hostname for SSL verification, and has to be input
appropriately by user), instead of just using ":<port>".
master
Aditya Manthramurthy 8 years ago committed by Harshavardhana
parent 95567c68bf
commit 6274727b71
  1. 2
      cmd/utils.go
  2. 19
      cmd/utils_test.go

@ -97,7 +97,7 @@ func splitNetPath(networkPath string) (netAddr, netPath string, err error) {
// `host:port` format. // `host:port` format.
func getLocalAddress(srvCmdConfig serverCmdConfig) string { func getLocalAddress(srvCmdConfig serverCmdConfig) string {
if !srvCmdConfig.isDistXL { if !srvCmdConfig.isDistXL {
return fmt.Sprintf(":%d", globalMinioPort) return srvCmdConfig.serverAddr
} }
for _, export := range srvCmdConfig.disks { for _, export := range srvCmdConfig.disks {
// Validates if remote disk is local. // Validates if remote disk is local.

@ -245,7 +245,8 @@ func TestLocalAddress(t *testing.T) {
// Test 2 - local address is everything. // Test 2 - local address is everything.
{ {
srvCmdConfig: serverCmdConfig{ srvCmdConfig: serverCmdConfig{
isDistXL: false, serverAddr: fmt.Sprintf(":%d", globalMinioPort),
isDistXL: false,
disks: []string{ disks: []string{
"/mnt/disk1", "/mnt/disk1",
"/mnt/disk2", "/mnt/disk2",
@ -268,6 +269,22 @@ func TestLocalAddress(t *testing.T) {
}, },
localAddr: "", localAddr: "",
}, },
// Test 4 - in case of FS mode, with SSL, the host
// name is specified in the --address option on the
// server command line.
{
srvCmdConfig: serverCmdConfig{
serverAddr: "play.minio.io:9000",
isDistXL: false,
disks: []string{
"/mnt/disk1",
"/mnt/disk2",
"/mnt/disk3",
"/mnt/disk4",
},
},
localAddr: "play.minio.io:9000",
},
} }
// Validates fetching local address. // Validates fetching local address.

Loading…
Cancel
Save