fixed linter

cmd/common.go

@@ -150,7 +150,7 @@ func getKubeConfigForUser(ctx *cli.Context, user string) (*api.Config, error) {
 	}
 
 	focusedServer := cf.FocusedServer()
-	kubeConfig, _ := focusedServer.KubeConfigs[fmt.Sprintf(kubeConfigKeyFormat, user, focusedServer.FocusedCluster())]
+	kubeConfig := focusedServer.KubeConfigs[fmt.Sprintf(kubeConfigKeyFormat, user, focusedServer.FocusedCluster())]
 	return kubeConfig, nil
 }
 

cmd/kubectl_token.go

@@ -485,6 +485,8 @@ func samlAuth(input *LoginInput, tlsConfig *tls.Config) (managementClient.Token,
 				// log error and use the token if login succeeds
 				customPrint(fmt.Errorf("DeleteToken: %v", err))
 			}
+			defer res.Body.Close()
+
 			return token, nil
 
 		case <-timeout.C:
@@ -502,12 +504,18 @@ func samlAuth(input *LoginInput, tlsConfig *tls.Config) (managementClient.Token,
 func getAuthProviders(server string) (map[string]string, error) {
 	authProviders := fmt.Sprintf(authProviderURL, server)
 	customPrint(authProviders)
+
 	response, err := request(http.MethodGet, authProviders, nil)
+	if err != nil {
+		return nil, err
+	}
+
 	data := map[string]interface{}{}
 	err = json.Unmarshal(response, &data)
 	if err != nil {
 		return nil, err
 	}
+
 	providers := map[string]string{}
 	i := 0
 	for _, value := range convert.ToMapSlice(data["data"]) {

cmd/multiclusterapp.go

@@ -345,6 +345,10 @@ func multiClusterAppLs(ctx *cli.Context) error {
 	}
 
 	collection, err := c.ManagementClient.MultiClusterApp.List(defaultListOpts(ctx))
+	if err != nil {
+		return err
+	}
+
 	writer := NewTableWriter([][]string{
 		{"ID", "ID"},
 		{"NAME", "App.Name"},

cmd/wait.go

@@ -62,13 +62,13 @@ func wait(ctx *cli.Context) error {
 	}
 
 	timeout := time.After(time.Duration(ctx.Int("timeout")) * time.Second)
-	every := time.Tick(1 * time.Second)
+	ticker := time.NewTicker(time.Second)
 
 	for {
 		select {
 		case <-timeout:
 			return fmt.Errorf("Timeout reached %v:%v transitioningMessage: %v", resource.Type, resource.ID, mapResource["transitioningMessage"])
-		case <-every:
+		case <-ticker.C:
 			err = c.ByID(resource, &mapResource)
 			if err != nil {
 				return err