please: trim the environmentURL

numtide · May 1, 2019 · 3829824 · 3829824
1 parent 267470b
commit 3829824
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/command/please.go b/command/please.go
@@ -110,16 +110,16 @@ func CmdPlease(c *cli.Context) (err error) {
 			EnvironmentURL: refString(environmentURL),
 			//AutoInactive: refBool(true),
 		})
-		if err != nil {
-			log.Println("updateStatus:", err)
-		}
 		return err
 	}
 
 	// Start deploy script
 	err = cmd.Start()
 	if err != nil {
-		updateStatus(StateError, "")
+		err2 := updateStatus(StateError, "")
+		if err2 != nil {
+			log.Println("updateStatus:", err)
+		}
 		return err
 	}
 
@@ -132,13 +132,16 @@ func CmdPlease(c *cli.Context) (err error) {
 	// Wait on the deploy to finish
 	err = cmd.Wait()
 	if err != nil {
-		updateStatus(StateFailure, "")
+		err2 := updateStatus(StateFailure, "")
+		if err2 != nil {
+			log.Println("updateStatus:", err)
+		}
 		return err
 	}
 
 	// Success!
 	out := strings.SplitN(stdout.String(), "\n", 2)
-	environmentURL := out[0]
+	environmentURL := strings.TrimSpace(out[0])
 	err = updateStatus(StateSuccess, environmentURL)
 	return err
 }