Merge pull request #177 from superfly/fixes-for-list-apps-auth

Fixes for err no auth crashes

.gitignore

@@ -30,3 +30,4 @@ docstrings/gen.go
 /.idea/misc.xml
 /.idea/modules.xml
 /.idea/vcs.xml
+.vscode/launch.json

api/client.go

@@ -33,9 +33,6 @@ type Client struct {
 }
 
 func NewClient(accessToken string, version string) *Client {
-	if accessToken == "" {
-		return nil, errors.New("No api access token available. Please login with 'flyctl auth signup' or 'flyctl auth login'")
-	}
 
 	httpClient, _ := newHTTPClient()
 

cmd/command.go

@@ -3,14 +3,15 @@ package cmd
 import (
 	"context"
 	"fmt"
-	"github.com/superfly/flyctl/cmdctx"
 	"io"
 	"os"
 	"os/signal"
 	"path"
 	"path/filepath"
 	"syscall"
 
+	"github.com/superfly/flyctl/cmdctx"
+
 	"github.com/spf13/cobra"
 	"github.com/spf13/viper"
 	"github.com/superfly/flyctl/flyctl"
@@ -207,9 +208,6 @@ func requireSession(cmd *Command) Initializer {
 			if !ctx.Client.Authenticated() {
 				return client.ErrNoAuthToken
 			}
-
-			// TODO: prompt to begin auth
-
 			return nil
 		},
 	}

cmd/list.go

@@ -2,10 +2,11 @@ package cmd
 
 import (
 	"fmt"
-	"github.com/superfly/flyctl/cmdctx"
 	"os"
 	"strings"
 
+	"github.com/superfly/flyctl/cmdctx"
+
 	"github.com/spf13/cobra"
 	"github.com/superfly/flyctl/docstrings"
 )
@@ -23,7 +24,7 @@ func newListCommand() *Command {
 	}
 
 	laks := docstrings.Get("list.apps")
-	listAppsCmd := BuildCommand(listCmd, runListApps, laks.Usage, laks.Short, laks.Long, os.Stdout)
+	listAppsCmd := BuildCommand(listCmd, runListApps, laks.Usage, laks.Short, laks.Long, os.Stdout, requireSession)
 
 	listAppsCmd.AddStringFlag(StringFlagOpts{
 		Name:        "org",
@@ -38,7 +39,7 @@ func newListCommand() *Command {
 	})
 
 	loks := docstrings.Get("list.orgs")
-	BuildCommand(listCmd, runListOrgs, loks.Usage, loks.Short, loks.Long, os.Stdout)
+	BuildCommand(listCmd, runListOrgs, loks.Usage, loks.Short, loks.Long, os.Stdout, requireSession)
 
 	return listCmd
 }