Revert "For analytics records, upload a singular product that was use…

…d for an authorized operation. This replaces the existing functionality that picked a random matching product from the credential" This reverts commit 8575002.
apigee · Sep 26, 2024 · 0673c38 · 0673c38
1 parent 8575002
commit 0673c38
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 7 deletions.
diff --git a/analytics/record.go b/analytics/record.go
@@ -86,9 +86,13 @@ func (r Record) EnsureFields(authContext *auth.Context) Record {
 	r.ClientID = authContext.ClientID
 	r.Organization = authContext.Organization()
 	r.Environment = authContext.Environment()
+
 	r.GatewayFlowID = uuid.New().String()
-	r.APIProduct = authContext.AnalyticsProduct
 
+	// select arbitrary APIProduct
+	if len(authContext.APIProducts) > 0 {
+		r.APIProduct = authContext.APIProducts[0]
+	}
 	return r
 }
 

diff --git a/auth/context.go b/auth/context.go
@@ -47,7 +47,6 @@ type Context struct {
 	Scopes           []string
 	APIKey           string
 	CustomAttributes string
-	AnalyticsProduct string // A single product to attatch to analytics records based on matched operations.
 }
 
 // if claims can't be processed, returns error and sets no fields

diff --git a/product/manager.go b/product/manager.go
@@ -82,7 +82,6 @@ type AuthorizedOperation struct {
 	QuotaLimit    int64
 	QuotaInterval int64
 	QuotaTimeUnit string
-	APIProduct    string
 }
 
 // Authorize a request against API Products and its Operations

diff --git a/product/product.go b/product/product.go
@@ -318,7 +318,6 @@ func (p *APIProduct) authorize(authContext *auth.Context, api, path, method stri
 					QuotaLimit:    p.QuotaLimitInt,
 					QuotaInterval: p.QuotaIntervalInt,
 					QuotaTimeUnit: p.QuotaTimeUnit,
-					APIProduct:    p.Name,
 				}
 				// OperationConfig quota is an override
 				if oc.Quota != nil && oc.Quota.LimitInt > 0 {
@@ -354,7 +353,6 @@ func (p *APIProduct) authorize(authContext *auth.Context, api, path, method stri
 		QuotaLimit:    p.QuotaLimitInt,
 		QuotaInterval: p.QuotaIntervalInt,
 		QuotaTimeUnit: p.QuotaTimeUnit,
-		APIProduct:    p.Name,
 	})
 	hint = "    authorized\n"
 

diff --git a/product/product_test.go b/product/product_test.go
@@ -196,7 +196,6 @@ func TestAuthorize(t *testing.T) {
 			wantAuthOp: &AuthorizedOperation{
 				ID:         "Name [email protected]",
 				QuotaLimit: productsMap["Name 2"].QuotaLimitInt,
-				APIProduct: "Name 2",
 			},
 			wantHints: `Authorizing request:
 			environment: prod
@@ -256,7 +255,6 @@ func TestAuthorize(t *testing.T) {
 			wantAuthOp: &AuthorizedOperation{
 				ID:         "Name [email protected]",
 				QuotaLimit: productsMap["Name 1"].QuotaLimitInt,
-				APIProduct: "Name 1",
 			},
 			wantHints: `Authorizing request:
 			environment: prod