Don't spam log when hostname cannot resolve

Author: lordwelch

cmd/serve.go

@@ -212,6 +212,14 @@ func isAddressReachable(addr string) (bool, error) {
 	pinger.Timeout = 2 * time.Second
 	pinger.Count = 1
 
+	// We resolve first because a host that's currently off may not have a dns entry and we don't want to spam the log.
+	// We could display this in the web interface in the future instead of ignoring it.
+	pinger.ResolveTimeout = 2 * time.Second
+	err = pinger.Resolve()
+	if err != nil {
+		return false, nil
+	}
+
 	err = pinger.Run()
 	if err != nil {
 		return false, fmt.Errorf("error pinging: %v", err)