expires in seconds consistently and for keys create and apps genkey only

apigee · Dec 13, 2023 · 685dd23 · 685dd23
1 parent c88639f
commit 685dd23
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 30 deletions.
diff --git a/cmd/apps/createkey.go b/cmd/apps/createkey.go
@@ -34,8 +34,10 @@ var CreateKeyCmd = &cobra.Command{
 		return apiclient.SetApigeeOrg(org)
 	},
 	RunE: func(cmd *cobra.Command, args []string) (err error) {
-		if _, err = strconv.Atoi(expires); err != nil {
-			return fmt.Errorf("expires must be an integer: %v", err)
+		if expires != "" {
+			if _, err = strconv.Atoi(expires); err != nil {
+				return fmt.Errorf("expires must be an integer: %v", err)
+			}
 		}
 		_, err = apps.CreateKey(developerEmail, name, key, secret, apiProducts, scopes, expires, attrs)
 		return
@@ -52,7 +54,7 @@ func init() {
 	CreateKeyCmd.Flags().StringArrayVarP(&scopes, "scopes", "s",
 		[]string{}, "OAuth scopes")
 	CreateKeyCmd.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")
 	CreateKeyCmd.Flags().StringToStringVar(&attrs, "attrs",
 		nil, "Custom attributes")
 

diff --git a/cmd/apps/genkey.go b/cmd/apps/genkey.go
@@ -15,6 +15,9 @@
 package apps
 
 import (
+	"fmt"
+	"strconv"
+
 	"internal/apiclient"
 
 	"internal/client/apps"
@@ -31,6 +34,12 @@ var GenKeyCmd = &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.GenerateKey(name, devID, apiProducts, callback, expires, scopes)
 		return
 	},
@@ -42,9 +51,9 @@ func init() {
 	GenKeyCmd.Flags().StringVarP(&name, "name", "n",
 		"", "Name of the developer app")
 	GenKeyCmd.Flags().StringVarP(&devID, "devid", "d",
-		"", "Developer Id")
+		"", "The developer's id or email")
 	GenKeyCmd.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")
 	GenKeyCmd.Flags().StringVarP(&callback, "callback", "c",
 		"", "The callbackUrl is used by OAuth")
 	GenKeyCmd.Flags().StringArrayVarP(&apiProducts, "prods", "p",

diff --git a/cmd/apps/updatekey.go b/cmd/apps/updatekey.go
@@ -15,9 +15,6 @@
 package apps
 
 import (
-	"fmt"
-	"strconv"
-
 	"internal/apiclient"
 
 	"internal/client/apps"
@@ -34,30 +31,22 @@ var UpdateKeyCmd = &cobra.Command{
 		return apiclient.SetApigeeOrg(org)
 	},
 	RunE: func(cmd *cobra.Command, args []string) (err error) {
-		if _, err = strconv.Atoi(expires); err != nil {
-			return fmt.Errorf("expires must be an integer: %v", err)
-		}
-		_, err = apps.UpdateKey(developerEmail, name, key, secret, apiProducts, scopes, expires, attrs)
+		_, err = apps.UpdateKey(developerEmail, name, key, secret, apiProducts, scopes, attrs)
 		return
 	},
 }
 
 func init() {
 	UpdateKeyCmd.Flags().StringVarP(&key, "key", "k",
 		"", "Developer app consumer key")
-	UpdateKeyCmd.Flags().StringVarP(&secret, "secret", "r",
-		"", "Developer app consumer secret")
 	UpdateKeyCmd.Flags().StringArrayVarP(&apiProducts, "prods", "p",
 		[]string{}, "A list of api products")
 	UpdateKeyCmd.Flags().StringArrayVarP(&scopes, "scopes", "s",
 		[]string{}, "OAuth scopes")
-	UpdateKeyCmd.Flags().StringVarP(&expires, "expires", "x",
-		"", "A setting, in milliseconds, for the lifetime of the consumer key")
 	UpdateKeyCmd.Flags().StringToStringVar(&attrs, "attrs",
 		nil, "Custom attributes")
 
 	_ = UpdateKeyCmd.MarkFlagRequired("name")
 	_ = UpdateKeyCmd.MarkFlagRequired("key")
-	_ = UpdateKeyCmd.MarkFlagRequired("secret")
 	_ = UpdateKeyCmd.MarkFlagRequired("prods")
 }
diff --git a/internal/client/apps/keys.go b/internal/client/apps/keys.go
@@ -44,11 +44,10 @@ func CreateKey(developerEmail string, appID string, consumerKey string, consumer
 	key = append(key, "\"consumerSecret\":\""+consumerSecret+"\"")
 
 	if expires != "" {
-		key = append(key, "\"expiresAt\":\""+expires+"\"")
+		key = append(key, "\"expiresInSeconds\":\""+expires+"\"")
 	}
 
 	payload := "{" + strings.Join(key, ",") + "}"
-
 	u.Path = path.Join(u.Path, apiclient.GetApigeeOrg(), "developers", developerEmail, "apps", appID, "keys")
 
 	if len(apiProducts) > 0 {
@@ -62,9 +61,8 @@ func CreateKey(developerEmail string, appID string, consumerKey string, consumer
 
 	// 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)
-		respBody, err = UpdateKeyProducts(developerEmail, appID, consumerKey, apiProducts, expires)
-		apiclient.ClientPrintHttpResponse.Set(apiclient.GetCmdPrintHttpResponseSetting())
+		apiclient.ClientPrintHttpResponse.Set(true)
+		respBody, err = UpdateKeyProducts(developerEmail, appID, consumerKey, apiProducts, scopes)
 	}
 
 	return respBody, err
@@ -88,7 +86,7 @@ func GetKey(developerEmail string, appID string, key string) (respBody []byte, e
 
 // UpdateKey
 func UpdateKey(developerEmail string, appID string, consumerKey string, consumerSecret string,
-	apiProducts []string, scopes []string, expires string, attrs map[string]string) (respBody []byte, err error) {
+	apiProducts []string, scopes []string, attrs map[string]string) (respBody []byte, err error) {
 	u, _ := url.Parse(apiclient.BaseURL)
 
 	key := []string{}
@@ -101,10 +99,6 @@ func UpdateKey(developerEmail string, appID string, consumerKey string, consumer
 		key = append(key, "\"scopes\":[\""+getArrayStr(scopes)+"\"]")
 	}
 
-	if expires != "" {
-		key = append(key, "\"expiresAt\":\""+expires+"\"")
-	}
-
 	if len(attrs) > 0 {
 		attributes := []string{}
 		for keyattr, value := range attrs {
@@ -128,13 +122,14 @@ func UpdateKey(developerEmail string, appID string, consumerKey string, consumer
 	return respBody, err
 }
 
-func UpdateKeyProducts(developerEmail string, appID string, consumerKey string, apiProducts []string, expires string) (respBody []byte, err error) {
+func UpdateKeyProducts(developerEmail string, appID string, consumerKey string, apiProducts []string, scopes []string) (respBody []byte, err error) {
 	u, _ := url.Parse(apiclient.BaseURL)
 
 	key := []string{}
 	key = append(key, "\"apiProducts\":[\""+getArrayStr(apiProducts)+"\"]")
-	if expires != "" {
-		key = append(key, "\"expiresAt\":\""+expires+"\"")
+
+	if len(scopes) > 0 {
+		key = append(key, "\"scopes\":[\""+getArrayStr(scopes)+"\"]")
 	}
 
 	payload := "{" + strings.Join(key, ",") + "}"