remove unused function

Emyrk · Emyrk · commit b08fc44864f9 · 2023-01-31T16:05:05.000-06:00
diff --git a/coderd/authzquery/organization.go b/coderd/authzquery/organization.go
@@ -10,14 +10,6 @@ import (
 	"github.com/coder/coder/coderd/rbac"
 )
 
-func (q *AuthzQuerier) GetAllOrganizationMembers(ctx context.Context, organizationID uuid.UUID) ([]database.User, error) {
-	// TODO: @emyrk this is returned by the template ACL api endpoint. These users are full database.Users, which is
-	// problematic since it bypasses the rbac.ResourceUser resource. We should probably return a organizationMember or
-	// restricted user type here instead. The returned user also is checking the User resource, whereas we might want to
-	// really check the OrganizationMember resource.
-	return authorizedFetchSet(q.authorizer, q.database.GetAllOrganizationMembers)(ctx, organizationID)
-}
-
 func (q *AuthzQuerier) GetGroupsByOrganizationID(ctx context.Context, organizationID uuid.UUID) ([]database.Group, error) {
 	return authorizedFetchSet(q.authorizer, q.database.GetGroupsByOrganizationID)(ctx, organizationID)
 }

Original file line number	Diff line number	Diff line change
`@@ -10,14 +10,6 @@ import (`
`10`	`10`	`"github.com/coder/coder/coderd/rbac"`
`11`	`11`	`)`
`12`	`12`
`13`		`-func (q *AuthzQuerier) GetAllOrganizationMembers(ctx context.Context, organizationID uuid.UUID) ([]database.User, error) {`
`14`		`- // TODO: @emyrk this is returned by the template ACL api endpoint. These users are full database.Users, which is`
`15`		`- // problematic since it bypasses the rbac.ResourceUser resource. We should probably return a organizationMember or`
`16`		`- // restricted user type here instead. The returned user also is checking the User resource, whereas we might want to`
`17`		`- // really check the OrganizationMember resource.`
`18`		`- return authorizedFetchSet(q.authorizer, q.database.GetAllOrganizationMembers)(ctx, organizationID)`
`19`		`-}`
`20`		`-`
`21`	`13`	`func (q *AuthzQuerier) GetGroupsByOrganizationID(ctx context.Context, organizationID uuid.UUID) ([]database.Group, error) {`
`22`	`14`	`return authorizedFetchSet(q.authorizer, q.database.GetGroupsByOrganizationID)(ctx, organizationID)`
`23`	`15`	`}`