Skip to content

Commit

Permalink
Merge pull request #162 from gvengel/tls-server-name-errors
Browse files Browse the repository at this point in the history
Add error messaging when the --memcached.tls.server-name flag is required
  • Loading branch information
matthiasr authored Apr 12, 2023
2 parents 8276a0f + a56d030 commit 6affbba
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion cmd/memcached_exporter/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ import (
"net"
"net/http"
"os"
"strings"

"github.com/alecthomas/kingpin/v2"
"github.com/go-kit/log/level"
Expand Down Expand Up @@ -66,7 +67,13 @@ func main() {
if *serverName == "" {
*serverName, _, err = net.SplitHostPort(*address)
if err != nil {
level.Error(logger).Log("msg", "Error running HTTP server", "err", err)
if strings.Contains(*address, "/") {
level.Error(logger).Log("msg",
"If --memcached.tls.enable is set and --memcached.address is a unix socket, "+
"you must also specify --memcached.tls.server-name")
} else {
level.Error(logger).Log("msg", "Error parsing memcached address", "err", err)
}
os.Exit(1)
}
}
Expand Down

0 comments on commit 6affbba

Please sign in to comment.