From d7cf76e77238faaeccb685760ad25018672e1683 Mon Sep 17 00:00:00 2001
From: Gyu-Ho Lee <gyuhox@gmail.com>
Date: Wed, 9 Aug 2017 13:35:09 -0700
Subject: [PATCH] embed: warns about empty hosts in advertise urls

Signed-off-by: Gyu-Ho Lee <gyuhox@gmail.com>
---
 embed/config.go | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/embed/config.go b/embed/config.go
index 2fb2a3280e75..4f85935f4a43 100644
--- a/embed/config.go
+++ b/embed/config.go
@@ -299,6 +299,14 @@ func (cfg *Config) Validate() error {
 	if err := checkBindURLs(cfg.ListenMetricsUrls); err != nil {
 		return err
 	}
+	if err := checkHostURLs(cfg.APUrls); err != nil {
+		// TODO: return err in v3.4
+		plog.Warning(err)
+	}
+	if err := checkHostURLs(cfg.ACUrls); err != nil {
+		// TODO: return err in v3.4
+		plog.Warning(err)
+	}
 
 	// Check if conflicting flags are passed.
 	nSet := 0
@@ -484,3 +492,16 @@ func checkBindURLs(urls []url.URL) error {
 	}
 	return nil
 }
+
+func checkHostURLs(urls []url.URL) error {
+	for _, url := range urls {
+		host, _, err := net.SplitHostPort(url.Host)
+		if err != nil {
+			return err
+		}
+		if host == "" {
+			return fmt.Errorf("unexpected empty host (%s)", url.String())
+		}
+	}
+	return nil
+}