summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSamantaz Fox <coding@samantaz.fr>2021-09-16 11:36:22 +0200
committerGitHub <noreply@github.com>2021-09-16 09:36:22 +0000
commite655af251c4c738b3ec3c0497d165deb4c781017 (patch)
tree79028262b14d6ec58b8c31b2101e9f022b72b112
parent5054510d15d58f4f72e4c004384a90ca61e08c63 (diff)
downloadinvidious-e655af251c4c738b3ec3c0497d165deb4c781017.tar.gz
invidious-e655af251c4c738b3ec3c0497d165deb4c781017.tar.bz2
invidious-e655af251c4c738b3ec3c0497d165deb4c781017.zip
Try not to log search queries (#2362)
-rw-r--r--src/invidious/helpers/logger.cr14
1 files changed, 13 insertions, 1 deletions
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