public/grafana
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore(authz): contextualize the authz logger (#106078)

Browse Source
This commit is contained in:
Jean-Philippe Quéméner
2025-05-27 16:28:58 +02:00
committed by GitHub
parent f511b313e6
commit 9a565ff46e
1 changed files with 16 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/services/authz/rbac/service.go
+16 -16
View File
@@ -107,15 +107,15 @@ func NewService(
func (s *Service) Check(ctx context.Context, req *authzv1.CheckRequest) (*authzv1.CheckResponse, error) {
ctx, span := s.tracer.Start(ctx, "authz_direct_db.service.Check")
defer span.End()
ctxLogger := s.logger.FromContext(ctx)
ctxLogger := s.logger.FromContext(ctx).New(
"subject", req.GetSubject(),
"namespace", req.GetNamespace(),
"group", req.GetGroup(),
"resource", req.GetResource(),
"verb", req.GetVerb(),
)
defer func(start time.Time) {
ctxLogger.Debug("Check execution time",
"subject", req.GetSubject(),
"namespace", req.GetNamespace(),
"group", req.GetGroup(),
"resource", req.GetResource(),
"verb", req.GetVerb(),
"duration", time.Since(start).Milliseconds())
ctxLogger.Debug("Check execution time", "duration", time.Since(start).Milliseconds())
}(time.Now())
deny := &authzv1.CheckResponse{Allowed: false}
@@ -184,15 +184,15 @@ func (s *Service) Check(ctx context.Context, req *authzv1.CheckRequest) (*authzv
func (s *Service) List(ctx context.Context, req *authzv1.ListRequest) (*authzv1.ListResponse, error) {
ctx, span := s.tracer.Start(ctx, "authz_direct_db.service.List")
defer span.End()
ctxLogger := s.logger.FromContext(ctx)
ctxLogger := s.logger.FromContext(ctx).New(
"subject", req.GetSubject(),
"namespace", req.GetNamespace(),
"group", req.GetGroup(),
"resource", req.GetResource(),
"verb", req.GetVerb(),
)
defer func(start time.Time) {
ctxLogger.Debug("List execution time",
"subject", req.GetSubject(),
"namespace", req.GetNamespace(),
"group", req.GetGroup(),
"resource", req.GetResource(),
"verb", req.GetVerb(),
"duration", time.Since(start).Milliseconds())
ctxLogger.Debug("List execution time", "duration", time.Since(start).Milliseconds())
}(time.Now())
listReq, err := s.validateListRequest(ctx, req)