remove unnecessary function and inline assignment

Signed-off-by: Lyas Spiehler <[email protected]>

prober/handler.go

@@ -108,10 +108,8 @@ func Handler(w http.ResponseWriter, r *http.Request, c *config.Config, logger lo
 	}
 
 	if module.Prober == "tcp" && hostname != "" {
-		err = setTLSServerName(hostname, &module)
-		if err != nil {
-			http.Error(w, err.Error(), http.StatusBadRequest)
-			return
+		if module.TCP.TLSConfig.ServerName == "" {
+			module.TCP.TLSConfig.ServerName = hostname
 		}
 	}
 
@@ -145,15 +143,6 @@ func Handler(w http.ResponseWriter, r *http.Request, c *config.Config, logger lo
 	h.ServeHTTP(w, r)
 }
 
-func setTLSServerName(hostname string, module *config.Module) error {
-	// By creating a new hashmap and copying values there we
-	// ensure that the initial configuration remain intact.
-	if module.TCP.TLSConfig.ServerName == "" {
-		module.TCP.TLSConfig.ServerName = hostname
-	}
-	return nil
-}
-
 func setHTTPHost(hostname string, module *config.Module) error {
 	// By creating a new hashmap and copying values there we
 	// ensure that the initial configuration remain intact.