From e655af251c4c738b3ec3c0497d165deb4c781017 Mon Sep 17 00:00:00 2001
From: Samantaz Fox <coding@samantaz.fr>
Date: Thu, 16 Sep 2021 11:36:22 +0200
Subject: [PATCH] Try not to log search queries (#2362)

---
 src/invidious/helpers/logger.cr | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/src/invidious/helpers/logger.cr b/src/invidious/helpers/logger.cr
index 5d91a258..e2e50905 100644
--- a/src/invidious/helpers/logger.cr
+++ b/src/invidious/helpers/logger.cr
@@ -17,7 +17,19 @@ class Invidious::LogHandler < Kemal::BaseLogHandler
     elapsed_time = Time.measure { call_next(context) }
     elapsed_text = elapsed_text(elapsed_time)
 
-    info("#{context.response.status_code} #{context.request.method} #{context.request.resource} #{elapsed_text}")
+    # Default: full path with parameters
+    requested_url = context.request.resource
+
+    # Try not to log search queries passed as GET parameters during normal use
+    # (They will still be logged if log level is 'Debug' or 'Trace')
+    if @level > LogLevel::Debug && (
+         requested_url.downcase.includes?("search") || requested_url.downcase.includes?("q=")
+       )
+      # Log only the path
+      requested_url = context.request.path
+    end
+
+    info("#{context.response.status_code} #{context.request.method} #{requested_url} #{elapsed_text}")
 
     context
   end