[auth] Only write config on auth flow

Author: dilyevsky

cmd/auth.go

@@ -26,18 +26,22 @@ If your CLI is already authenticated this will return information about your ses
 		}
 
 		auth := config.NewAuthenticator(cfg)
-		if ok, err := auth.Check(); err != nil {
+		ok, err := auth.Check()
+		if err != nil {
 			fmt.Println(err)
 			os.Exit(1)
-		} else if ok {
+		}
+
+		if ok {
 			fmt.Println("Authenticated")
-		} else if !checkOnly {
-			fmt.Println("Authentication required. Opening browser...")
-			auth.Authenticate()
-		} else {
+			os.Exit(0)
+		} else if checkOnly { // If we're only checking, exit with an error.
 			fmt.Println("Invalid authentication")
+			os.Exit(1)
 		}
 
+		fmt.Println("Authentication required. Opening browser...")
+		auth.Authenticate()
 		if err := config.Store(cfg); err != nil {
 			fmt.Println(err)
 		}