Setting expires consistently for seconds or Apps and AppGroup Apps

cmd/appgroups/createkey.go

@@ -36,14 +36,18 @@ var CreateKeyCmd = &cobra.Command{
 		return apiclient.SetApigeeOrg(org)
 	},
 	RunE: func(cmd *cobra.Command, args []string) (err error) {
-		_, err = appgroups.CreateKey(name, appName, key, secret, strconv.Itoa(expiry), apiProducts, scopes, attrs)
+		if expires != "" {
+			if _, err = strconv.Atoi(expires); err != nil {
+				return fmt.Errorf("expires must be an integer: %v", err)
+			}
+		}
+		_, err = appgroups.CreateKey(name, appName, key, secret, expires, apiProducts, scopes, attrs)
 		return
 	},
 }
 
 var (
-	secret string
-	expiry int
+	secret  string
 )
 
 func init() {
@@ -55,8 +59,8 @@ func init() {
 		"", "Import an existing AppGroup app consumer key")
 	CreateKeyCmd.Flags().StringVarP(&secret, "secret", "r",
 		"", "Import an existing AppGroup app consumer secret")
-	CreateKeyCmd.Flags().IntVarP(&expiry, "expiry", "x",
-		-1, "Expiration time, in seconds, for the consumer key")
+	CreateKeyCmd.Flags().StringVarP(&expires, "expires", "x",
+		"", "A setting, in seconds, for the lifetime of the consumer key")
 	CreateKeyCmd.Flags().StringArrayVarP(&apiProducts, "prods", "p",
 		[]string{}, "A list of api products")
 	CreateKeyCmd.Flags().StringArrayVarP(&scopes, "scopes", "s",
@@ -66,4 +70,5 @@ func init() {
 
 	_ = CreateKeyCmd.MarkFlagRequired("name")
 	_ = CreateKeyCmd.MarkFlagRequired("app-name")
+	_ = CreateKeyCmd.MarkFlagRequired("prods")
 }

cmd/appgroups/crtapp.go

@@ -15,6 +15,9 @@
 package appgroups
 
 import (
+	"fmt"
+	"strconv"
+
 	"internal/apiclient"
 
 	"internal/client/appgroups"
@@ -31,6 +34,12 @@ var CreateAppCmd = &cobra.Command{
 		return apiclient.SetApigeeOrg(org)
 	},
 	RunE: func(cmd *cobra.Command, args []string) (err error) {
+		if expires != "" {
+			if _, err = strconv.Atoi(expires); err != nil {
+				return fmt.Errorf("expires must be an integer: %v", err)
+			}
+			expires += "000"
+		}
 		_, err = appgroups.CreateApp(name, appName, expires, callback, apiProducts, scopes, attrs)
 		return
 	},
@@ -47,7 +56,7 @@ func init() {
 	CreateAppCmd.Flags().StringVarP(&appName, "app-name", "",
 		"", "Name of the app")
 	CreateAppCmd.Flags().StringVarP(&expires, "expires", "x",
-		"", "A setting, in milliseconds, for the lifetime of the consumer key")
+		"", "A setting, in seconds, for the lifetime of the consumer key")
 	CreateAppCmd.Flags().StringVarP(&callback, "callback", "c",
 		"", "The callbackUrl is used by OAuth")
 	CreateAppCmd.Flags().StringArrayVarP(&apiProducts, "prods", "p",
@@ -59,4 +68,5 @@ func init() {
 
 	_ = CreateAppCmd.MarkFlagRequired("name")
 	_ = CreateAppCmd.MarkFlagRequired("app-name")
+	_ = CreateAppCmd.MarkFlagRequired("prods")
 }

cmd/appgroups/updateapp.go

@@ -15,6 +15,9 @@
 package appgroups
 
 import (
+	"fmt"
+	"strconv"
+
 	"internal/apiclient"
 
 	"internal/client/appgroups"
@@ -31,6 +34,12 @@ var UpdateAppCmd = &cobra.Command{
 		return apiclient.SetApigeeOrg(org)
 	},
 	RunE: func(cmd *cobra.Command, args []string) (err error) {
+		if expires != "" {
+			if _, err = strconv.Atoi(expires); err != nil {
+				return fmt.Errorf("expires must be an integer: %v", err)
+			}
+			expires += "000"
+		}
 		_, err = appgroups.UpdateApp(name, appName, expires, callback, apiProducts, scopes, attrs)
 		return
 	},
@@ -42,7 +51,7 @@ func init() {
 	UpdateAppCmd.Flags().StringVarP(&appName, "app-name", "",
 		"", "Name of the app")
 	UpdateAppCmd.Flags().StringVarP(&expires, "expires", "x",
-		"", "A setting, in milliseconds, for the lifetime of the consumer key")
+		"", "A setting, in seconds, for the lifetime of the consumer key")
 	UpdateAppCmd.Flags().StringVarP(&callback, "callback", "c",
 		"", "The callbackUrl is used by OAuth")
 	UpdateAppCmd.Flags().StringArrayVarP(&apiProducts, "prods", "p",

cmd/apps/crtapp.go

@@ -15,6 +15,9 @@
 package apps
 
 import (
+	"fmt"
+	"strconv"
+
 	"internal/apiclient"
 
 	"internal/client/apps"
@@ -31,6 +34,12 @@ var CreateCmd = &cobra.Command{
 		return apiclient.SetApigeeOrg(org)
 	},
 	RunE: func(cmd *cobra.Command, args []string) (err error) {
+		if expires != "" {
+			if _, err = strconv.Atoi(expires); err != nil {
+				return fmt.Errorf("expires must be an integer: %v", err)
+			}
+			expires += "000"
+		}
 		_, err = apps.Create(name, email, expires, callback, apiProducts, scopes, attrs)
 		return
 	},
@@ -48,7 +57,7 @@ func init() {
 	CreateCmd.Flags().StringVarP(&email, "email", "e",
 		"", "The developer's email or id")
 	CreateCmd.Flags().StringVarP(&expires, "expires", "x",
-		"", "A setting, in milliseconds, for the lifetime of the consumer key")
+		"", "A setting, in seconds, for the lifetime of the consumer key")
 	CreateCmd.Flags().StringVarP(&callback, "callback", "c",
 		"", "The callbackUrl is used by OAuth")
 	CreateCmd.Flags().StringArrayVarP(&apiProducts, "prods", "p",

cmd/apps/updateapp.go

@@ -15,6 +15,9 @@
 package apps
 
 import (
+	"fmt"
+	"strconv"
+
 	"internal/apiclient"
 
 	"internal/client/apps"
@@ -31,6 +34,12 @@ var UpdateCmd = &cobra.Command{
 		return apiclient.SetApigeeOrg(org)
 	},
 	RunE: func(cmd *cobra.Command, args []string) (err error) {
+		if expires != "" {
+			if _, err = strconv.Atoi(expires); err != nil {
+				return fmt.Errorf("expires must be an integer: %v", err)
+			}
+			expires += "000"
+		}
 		_, err = apps.Update(name, email, expires, callback, apiProducts, scopes, attrs)
 		return
 	},

cmd/apps/updatekey.go

@@ -31,7 +31,7 @@ var UpdateKeyCmd = &cobra.Command{
 		return apiclient.SetApigeeOrg(org)
 	},
 	RunE: func(cmd *cobra.Command, args []string) (err error) {
-		_, err = apps.UpdateKey(developerEmail, name, key, secret, apiProducts, scopes, attrs)
+		_, err = apps.UpdateKey(developerEmail, name, key, apiProducts, scopes, attrs)
 		return
 	},
 }

internal/client/appgroups/keys.go

@@ -45,6 +45,10 @@ func CreateKey(name string, appName string, consumerKey string, consumerSecret s
 		key = append(key, "\"consumerSecret\":\""+consumerSecret+"\"")
 	}
 
+	if expiresInSeconds != "" {
+		key = append(key, "\"expiresInSeconds\":\""+expiresInSeconds+"\"")
+	}
+
 	payload := "{" + strings.Join(key, ",") + "}"
 
 	u.Path = path.Join(u.Path, apiclient.GetApigeeOrg(), "appgroups", name, "apps", appName, "keys")
@@ -60,9 +64,8 @@ func CreateKey(name string, appName string, consumerKey string, consumerSecret s
 
 	// since the API does not support adding products when creating a key, use a second API call to add products
 	if len(apiProducts) > 0 {
-		apiclient.ClientPrintHttpResponse.Set(false)
+		apiclient.ClientPrintHttpResponse.Set(true)
 		respBody, err = UpdateKeyProducts(name, appName, consumerKey, apiProducts)
-		apiclient.ClientPrintHttpResponse.Set(apiclient.GetCmdPrintHttpResponseSetting())
 	}
 
 	return respBody, err

internal/client/apps/keys.go

@@ -85,7 +85,7 @@ func GetKey(developerEmail string, appID string, key string) (respBody []byte, e
 }
 
 // UpdateKey
-func UpdateKey(developerEmail string, appID string, consumerKey string, consumerSecret string,
+func UpdateKey(developerEmail string, appID string, consumerKey string,
 	apiProducts []string, scopes []string, attrs map[string]string) (respBody []byte, err error) {
 	u, _ := url.Parse(apiclient.BaseURL)
 
@@ -110,10 +110,6 @@ func UpdateKey(developerEmail string, appID string, consumerKey string, consumer
 
 	key = append(key, "\"consumerKey\":\""+consumerKey+"\"")
 
-	if consumerSecret != "" {
-		key = append(key, "\"consumerSecret\":\""+consumerSecret+"\"")
-	}
-
 	payload := "{" + strings.Join(key, ",") + "}"
 
 	u.Path = path.Join(u.Path, apiclient.GetApigeeOrg(), "developers", developerEmail, "apps", appID, "keys", consumerKey)