Merge pull request #103 from AskAlice/sectionctl-login-error-handling

sectionctl login keychain error handling

Author: Lindsay Holmwood

.gitignore

@@ -19,6 +19,7 @@ bin/
 build/
 dist/
 sectionctl
-
+.vscode/
+__debug_bin
 # Debug outputs
 sectionctl-debug-*.log

.vscode.example/launch.json

@@ -0,0 +1,86 @@
+{
+  "version": "0.2.0",
+  "configurations": [
+    {
+      "name": "deploy",
+      "type": "go",
+      "request": "launch",
+      "mode": "debug",
+      "cwd": "${workspaceFolder}/commands/testdata/deploy/api-routes-app",
+      "program": "${workspaceFolder}/sectionctl.go",
+      "args": ["deploy", "-a", "replaceMeWithYourAccountID", "-i", "replaceMeWithYourApplicationID"]
+    },
+    {
+      "name": "accounts/list",
+      "type": "go",
+      "request": "launch",
+      "mode": "debug",
+      "cwd": "${workspaceFolder}",
+      "program": "${workspaceFolder}/sectionctl.go",
+      "args": ["accounts", "list"]
+    },
+    {
+      "name": "apps/list",
+      "type": "go",
+      "request": "launch",
+      "mode": "debug",
+      "cwd": "${workspaceFolder}",
+      "program": "${workspaceFolder}/sectionctl.go",
+      "args": ["apps", "list", "-a", "replaceMeWithYourAccountID"]
+    },
+    {
+      "name": "apps/info",
+      "type": "go",
+      "request": "launch",
+      "mode": "debug",
+      "cwd": "${workspaceFolder}",
+      "program": "${workspaceFolder}/sectionctl.go",
+      "args": ["apps", "info", "-a", "replaceMeWithYourAccountID", "-i", "replaceMeWithYourApplicationID"]
+    },
+    {
+      "name": "apps/stacks",
+      "type": "go",
+      "request": "launch",
+      "mode": "debug",
+      "cwd": "${workspaceFolder}",
+      "program": "${workspaceFolder}/sectionctl.go",
+      "args": ["apps", "stacks"]
+    },
+    {
+      "name": "domains/list",
+      "type": "go",
+      "request": "launch",
+      "mode": "debug",
+      "cwd": "${workspaceFolder}",
+      "program": "${workspaceFolder}/sectionctl.go",
+      "args": ["domains", "list"]
+    },
+    {
+      "name": "logs",
+      "type": "go",
+      "request": "launch",
+      "mode": "debug",
+      "cwd": "${workspaceFolder}",
+      "program": "${workspaceFolder}/sectionctl.go",
+      "args": ["logs", "-a", "replaceMeWithYourAccountID", "-i", "replaceMeWithYourApplicationID"]
+    },
+    {
+      "name": "ps",
+      "type": "go",
+      "request": "launch",
+      "mode": "debug",
+      "cwd": "${workspaceFolder}",
+      "program": "${workspaceFolder}/sectionctl.go",
+      "args": ["ps", "-a", "replaceMeWithYourAccountID"]
+    },
+    {
+      "name": "login",
+      "type": "go",
+      "request": "launch",
+      "mode": "debug",
+      "cwd": "${workspaceFolder}",
+      "program": "${workspaceFolder}/sectionctl.go",
+      "args": ["login", "--section-token", "replaceMeWithYourSectionToken"]
+    }
+  ]
+}

commands/login.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"io"
 	"os"
+	"runtime"
 
 	"github.com/section/sectionctl/api"
 	"github.com/section/sectionctl/credentials"
@@ -21,6 +22,14 @@ func (c *LoginCmd) Run() (err error) {
 	if api.Token != "" {
 		err = credentials.Write(api.PrefixURI.Host, api.Token)
 		if err != nil {
+			if runtime.GOOS == "linux" || runtime.GOOS == "darwin" {
+				fmt.Printf("Unable to write credential.\n\nPlease run this command, and add it to your ~/.bashrc\n\nexport SECTION_TOKEN=%s\n",api.Token)
+				return nil
+			} else if runtime.GOOS == "windows" {
+				screenshot := "https://raw.githubusercontent.com/section/sectionctl/main/docs/section_token_control_panel.png"
+				fmt.Printf("Unable to write credential.\n\nPlease execute the following, add it to your Powershell profile, or add it to your environment variables in control panel: \nWith Powershell:\n$env:SECTION_TOKEN=\"%s\"\n\nWith CMD:\nset SECTION_TOKEN=%s\n\nWith control panel:\n%s",api.Token,api.Token,screenshot)
+				return nil
+			}
 			return fmt.Errorf("unable to write credential: %w", err)
 		}
 	} else {
@@ -30,7 +39,6 @@ func (c *LoginCmd) Run() (err error) {
 		}
 		api.Token = t
 	}
-
 	fmt.Print("\nValidating credentials...")
 	_, err = api.CurrentUser()
 	if err != nil {