reference global logrus (#274)

* reference logrus' global logger instead of passing and storing a logger reference everywhere

* always directly use global logrus logger instead of referencing an instance

* test suites should also directly use the global logrus logger

* rename gin logging function to clarify that it's middleware

* correct comments which erroneously referenced removed logger parameter

* setting log level for tests now uses logrus' exported type instead of the string value, to guarantee error isn't possible

1 files changed, 4 insertions, 6 deletions

diff --git a/internal/db/bundb/trace.go b/internal/db/bundb/trace.go
index 82b50c058..b13adf1bf 100644
--- a/internal/db/bundb/trace.go
+++ b/internal/db/bundb/trace.go
@@ -27,26 +27,24 @@ import (
 	"github.com/uptrace/bun"
 )
 
-func newDebugQueryHook(log *logrus.Logger) bun.QueryHook {
+func newDebugQueryHook() bun.QueryHook {
 	return &debugQueryHook{
-		log: log,
 	}
 }
 
 // debugQueryHook implements bun.QueryHook
 type debugQueryHook struct {
-	log *logrus.Logger
 }
 
-func (q *debugQueryHook) BeforeQuery(ctx context.Context, event *bun.QueryEvent) context.Context {
+func (q *debugQueryHook) BeforeQuery(ctx context.Context, _ *bun.QueryEvent) context.Context {
 	// do nothing
 	return ctx
 }
 
 // AfterQuery logs the time taken to query, the operation (select, update, etc), and the query itself as translated by bun.
-func (q *debugQueryHook) AfterQuery(ctx context.Context, event *bun.QueryEvent) {
+func (q *debugQueryHook) AfterQuery(_ context.Context, event *bun.QueryEvent) {
 	dur := time.Since(event.StartTime).Round(time.Microsecond)
-	l := q.log.WithFields(logrus.Fields{
+	l := logrus.WithFields(logrus.Fields{
 		"duration":  dur,
 		"operation": event.Operation(),
 	})