Merge pull request #104 from ConductorOne/bugfix/wrap_error_w_permission_code

agustin-conductor · web-flow · commit 3fe699376359 · 2025-09-16T09:13:36.000-03:00
[BB-1313] wrap error w permission denied code
diff --git a/pkg/connector/connector.go b/pkg/connector/connector.go
@@ -180,7 +180,8 @@ func (gh *GitHub) Validate(ctx context.Context) (annotations.Annotations, error)
 		membership, _, err := gh.client.Organizations.GetOrgMembership(ctx, "", o)
 		if err != nil {
 			if filterOrgs {
-				return nil, fmt.Errorf("can't get authenticated user on the %s organization: %w", o, err)
+				err := fmt.Errorf("can't get authenticated user on the %s organization: %w", o, err)
+				return nil, uhttp.WrapErrors(codes.PermissionDenied, "github-connector: credentials validation failed", err)
 			}
 			continue
 		}

Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,8 @@ func (gh *GitHub) Validate(ctx context.Context) (annotations.Annotations, error)`
`180`	`180`	`membership, _, err := gh.client.Organizations.GetOrgMembership(ctx, "", o)`
`181`	`181`	`if err != nil {`
`182`	`182`	`if filterOrgs {`
`183`		`- return nil, fmt.Errorf("can't get authenticated user on the %s organization: %w", o, err)`
	`183`	`+ err := fmt.Errorf("can't get authenticated user on the %s organization: %w", o, err)`
	`184`	`+ return nil, uhttp.WrapErrors(codes.PermissionDenied, "github-connector: credentials validation failed", err)`
`184`	`185`	`}`
`185`	`186`	`continue`
`186`	`187`	`}`