From 311f9a1d000418c04fe3eaadec1ba1286839fe10 Mon Sep 17 00:00:00 2001 From: James Read Date: Tue, 24 Oct 2023 05:43:38 +0100 Subject: [PATCH] feature: Reduce debug log spam in ACL/getDashboardComponents (#176) --- internal/acl/acl.go | 8 ++++---- internal/grpcapi/grpcApi.go | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/internal/acl/acl.go b/internal/acl/acl.go index 8aa7222..979d37b 100644 --- a/internal/acl/acl.go +++ b/internal/acl/acl.go @@ -25,7 +25,7 @@ func IsAllowedExec(cfg *config.Config, user *AuthenticatedUser, action *config.A "User": user.Username, "Action": action.Title, "ACL": acl.Name, - }).Debug("isAllowedExec - Matched ACL") + }).Trace("isAllowedExec - Matched ACL") return true } @@ -34,7 +34,7 @@ func IsAllowedExec(cfg *config.Config, user *AuthenticatedUser, action *config.A log.WithFields(log.Fields{ "User": user.Username, "Action": action.Title, - }).Debug("isAllowedExec - No ACLs matched") + }).Trace("isAllowedExec - No ACLs matched") return cfg.DefaultPermissions.Exec } @@ -47,7 +47,7 @@ func IsAllowedView(cfg *config.Config, user *AuthenticatedUser, action *config.A "User": user.Username, "Action": action.Title, "ACL": acl.Name, - }).Debug("isAllowedView - Matched ACL") + }).Trace("isAllowedView - Matched ACL") return true } @@ -56,7 +56,7 @@ func IsAllowedView(cfg *config.Config, user *AuthenticatedUser, action *config.A log.WithFields(log.Fields{ "User": user.Username, "Action": action.Title, - }).Debug("isAllowedView - No ACLs matched") + }).Trace("isAllowedView - No ACLs matched") return cfg.DefaultPermissions.View } diff --git a/internal/grpcapi/grpcApi.go b/internal/grpcapi/grpcApi.go index 87be267..8b253d7 100644 --- a/internal/grpcapi/grpcApi.go +++ b/internal/grpcapi/grpcApi.go @@ -115,7 +115,7 @@ func (api *oliveTinAPI) GetDashboardComponents(ctx ctx.Context, req *pb.GetDashb log.Warn("Zero actions found - check that you have some actions defined, with a view permission") } - log.Debugf("GetDashboardComponents: %v", res) + log.Tracef("GetDashboardComponents: %v", res) return res, nil }