Trim the "@-" suffix from resource names before calling the List*Revi…

…sions APIs. (#1218)
apigee · Aug 14, 2023 · 82fd89e · 82fd89e
1 parent bce4232
commit 82fd89e
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/pkg/visitor/list.go b/pkg/visitor/list.go
@@ -17,6 +17,7 @@ package visitor
 import (
 	"context"
 	"fmt"
+	"strings"
 
 	"github.com/apigee/registry/cmd/registry/compress"
 	"github.com/apigee/registry/gapic"
@@ -132,7 +133,8 @@ func ListDeploymentRevisions(ctx context.Context,
 	filter string,
 	handler DeploymentHandler) error {
 	it := client.ListApiDeploymentRevisions(ctx, &rpc.ListApiDeploymentRevisionsRequest{
-		Name:     name.String(),
+		// "@-" indicates a collection of revisions, but we only want to send the resource name to the List RPC.
+		Name:     strings.TrimSuffix(name.String(), "@-"),
 		PageSize: pageSize,
 		Filter:   filter,
 	})
@@ -234,7 +236,8 @@ func ListSpecRevisions(ctx context.Context,
 	getContents bool,
 	handler SpecHandler) error {
 	it := client.ListApiSpecRevisions(ctx, &rpc.ListApiSpecRevisionsRequest{
-		Name:     name.String(),
+		// "@-" indicates a collection of revisions, but we only want to send the resource name to the List RPC.
+		Name:     strings.TrimSuffix(name.String(), "@-"),
 		PageSize: pageSize,
 		Filter:   filter,
 	})