From 3f969f20f2b35dc2ed8fc125b1fbebd5bf3fb831 Mon Sep 17 00:00:00 2001 From: Michael Schubert Date: Thu, 15 Dec 2016 14:22:26 +0100 Subject: [PATCH] prog/main: use flags.app.port for default target User can control the webserver listen address with `-app.http.address`. If they do, `scope --mode probe ...` should use the port part of address in the default target and not ':4040'. Otherwise we encounter a `dial tcp 127.0.0.1:4040: getsockopt: connection refused` error and `scope-probe` cannot report. Signed-off-by: Michael Schubert --- prog/main.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/prog/main.go b/prog/main.go index ac1ca13a7..1aeaef1b5 100644 --- a/prog/main.go +++ b/prog/main.go @@ -344,7 +344,7 @@ func main() { flags.probe.noApp = *noApp || *probeOnly // Special case for #1191, check listen address is well formed - _, _, err := net.SplitHostPort(flags.app.listen) + _, port, err := net.SplitHostPort(flags.app.listen) if err != nil { log.Fatalf("Invalid value for -app.http.address: %v", err) } @@ -365,7 +365,7 @@ func main() { args = append(args, defaultServiceHost) } } else if !flags.probe.noApp { - args = append(args, fmt.Sprintf("localhost:%d", xfer.AppPort)) + args = append(args, fmt.Sprintf("localhost:%s", port)) } args = append(args, flag.Args()...) if !dryRun {