Improved reported error on invalid XDS_AGENT_URL
authorSebastien Douheret <sebastien.douheret@iot.bzh>
Thu, 7 Dec 2017 16:45:29 +0000 (17:45 +0100)
committerSebastien Douheret <sebastien.douheret@iot.bzh>
Thu, 7 Dec 2017 16:45:29 +0000 (17:45 +0100)
Specifically when AGENT_URL is set to SERVER_URL and consequently connection failed with "Failed to get device ID" error.

main.go

diff --git a/main.go b/main.go
index ad237af..bf03375 100644 (file)
--- a/main.go
+++ b/main.go
@@ -328,15 +328,22 @@ func XdsConnInit(ctx *cli.Context) error {
                CsrfDisable:         true,
                LogOut:              Log.Out,
                LogPrefix:           "XDSAGENT: ",
-               LogLevel:            common.HTTPLogLevelWarning,
+               LogLevel:            common.HTTPLogLevelDebug,
        }
 
        HTTPCli, err = common.HTTPNewClient(agentURL, conf)
        if err != nil {
                errmsg := err.Error()
-               if m, err := regexp.MatchString("Get http.?://", errmsg); m && err == nil {
+               m, err := regexp.MatchString("Get http.?://", errmsg)
+               if (m && err == nil) || strings.Contains(errmsg, "Failed to get device ID") {
                        i := strings.LastIndex(errmsg, ":")
-                       errmsg = "Cannot connection to " + agentURL + errmsg[i:]
+                       newErr := "Cannot connection to " + agentURL
+                       if i > 0 {
+                               newErr += " (" + strings.TrimSpace(errmsg[i+1:]) + ")"
+                       } else {
+                               newErr += " (" + strings.TrimSpace(errmsg) + ")"
+                       }
+                       errmsg = newErr
                }
                return cli.NewExitError(errmsg, 1)
        }