fix: Do not check for pubsub.topics.get on initialization (#1853)

jalseth · web-flow · commit 9e37c195cc5a · 2023-12-03T06:25:29.000-05:00
This permission is not strictly needed and is not included in the pubsub.publisher
role.

Signed-off-by: James Alseth &lt;james@jalseth.me&gt;
diff --git a/pkg/pubsub/gcp/publisher.go b/pkg/pubsub/gcp/publisher.go
@@ -45,7 +45,6 @@ var (
 	// Minimal set of permissions needed to check if the server can publish to the configured topic.
 	// https://cloud.google.com/pubsub/docs/access-control#required_permissions
 	requiredIAMPermissions = []string{
-		"pubsub.topics.get",
 		"pubsub.topics.publish",
 	}
 )

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,6 @@ var (`
`45`	`45`	`// Minimal set of permissions needed to check if the server can publish to the configured topic.`
`46`	`46`	`// https://cloud.google.com/pubsub/docs/access-control#required_permissions`
`47`	`47`	`requiredIAMPermissions = []string{`
`48`		`- "pubsub.topics.get",`
`49`	`48`	`"pubsub.topics.publish",`
`50`	`49`	`}`
`51`	`50`	`)`