From 15b9a5978685b7311ceed5a817b2e28bb81cf653 Mon Sep 17 00:00:00 2001
From: Colin Adler <colin1adler@gmail.com>
Date: Fri, 23 Sep 2022 15:21:56 -0500
Subject: [PATCH] chore: only trace rbac.Filter (#4177)

---
 coderd/rbac/authz.go   | 9 ---------
 coderd/rbac/partial.go | 3 ---
 2 files changed, 12 deletions(-)

diff --git a/coderd/rbac/authz.go b/coderd/rbac/authz.go
index 591df4bf5b..3bc37d0564 100644
--- a/coderd/rbac/authz.go
+++ b/coderd/rbac/authz.go
@@ -131,9 +131,6 @@ func (a RegoAuthorizer) ByRoleName(ctx context.Context, subjectID string, roleNa
 // Authorize allows passing in custom Roles.
 // This is really helpful for unit testing, as we can create custom roles to exercise edge cases.
 func (a RegoAuthorizer) Authorize(ctx context.Context, subjectID string, roles []Role, scope Role, action Action, object Object) error {
-	ctx, span := tracing.StartSpan(ctx)
-	defer span.End()
-
 	input := map[string]interface{}{
 		"subject": authSubject{
 			ID:    subjectID,
@@ -179,9 +176,6 @@ func (a RegoAuthorizer) Authorize(ctx context.Context, subjectID string, roles [
 // Prepare will partially execute the rego policy leaving the object fields unknown (except for the type).
 // This will vastly speed up performance if batch authorization on the same type of objects is needed.
 func (RegoAuthorizer) Prepare(ctx context.Context, subjectID string, roles []Role, scope Role, action Action, objectType string) (*PartialAuthorizer, error) {
-	ctx, span := tracing.StartSpan(ctx)
-	defer span.End()
-
 	auth, err := newPartialAuthorizer(ctx, subjectID, roles, scope, action, objectType)
 	if err != nil {
 		return nil, xerrors.Errorf("new partial authorizer: %w", err)
@@ -191,9 +185,6 @@ func (RegoAuthorizer) Prepare(ctx context.Context, subjectID string, roles []Rol
 }
 
 func (a RegoAuthorizer) PrepareByRoleName(ctx context.Context, subjectID string, roleNames []string, scope Scope, action Action, objectType string) (PreparedAuthorized, error) {
-	ctx, span := tracing.StartSpan(ctx)
-	defer span.End()
-
 	roles, err := RolesByNames(roleNames)
 	if err != nil {
 		return nil, err
diff --git a/coderd/rbac/partial.go b/coderd/rbac/partial.go
index 8cdc330352..637440e121 100644
--- a/coderd/rbac/partial.go
+++ b/coderd/rbac/partial.go
@@ -28,9 +28,6 @@ type PartialAuthorizer struct {
 var _ PreparedAuthorized = (*PartialAuthorizer)(nil)
 
 func (pa *PartialAuthorizer) Authorize(ctx context.Context, object Object) error {
-	ctx, span := tracing.StartSpan(ctx)
-	defer span.End()
-
 	if pa.alwaysTrue {
 		return nil
 	}