diff --git a/ops/metrics/alloy/alloy-apps.alloy b/ops/metrics/alloy/alloy-apps.alloy index 9d0b931b..709d621f 100644 --- a/ops/metrics/alloy/alloy-apps.alloy +++ b/ops/metrics/alloy/alloy-apps.alloy @@ -172,6 +172,26 @@ loki.process "scrub_credentials" { expression = "://[^:@/ ]+:([^@ ]+)@" replace = "***" } + + // ── Секреты в query-строке (#3354) ───────────────────────────────────────── + // Прод-факт (#3154): uvicorn пишет в access-log ПОЛНЫЙ путь с query, и туда + // уезжал `?secret=<64 hex>` вебхука GlitchTip. Приложение чистит это у себя + // (tradein-mvp/backend/app/core/log_scrub.py), здесь — второй слой на случай + // строки, пришедшей мимо фильтра: другой процесс, sidecar, будущий логгер. + // + // Множество имён держим ОДИНАКОВЫМ с log_scrub.py. Префикс `[\w.-]*` перед + // альтернацией — ради суффиксных имён (`client_secret`, `refresh_token`, + // `webhook_secret`); значение обрывается на `&`, пробеле или кавычке, потому + // что access-строка uvicorn обрамляет запрос кавычками. + // + // Группа захвата стоит на ЗНАЧЕНИИ, а не на имени параметра: Alloy заменяет + // содержимое групп, а не весь совпавший фрагмент (ровно как у DSN выше — там + // группа на пароле, поэтому пользователь и хост уцелевают). Обернуть группой + // `?secret=` значило бы затереть имя и оставить сам секрет. + stage.replace { + expression = "(?i)[?&][\\w.-]*(?:secret|token|api[-_]?key|apikey|access[-_]?token|password|signature|sig)=([^&\\s\"'<>]+)" + replace = "***" + } } loki.relabel "journal" { diff --git a/ops/metrics/alloy/alloy-infra.alloy b/ops/metrics/alloy/alloy-infra.alloy index 1b688957..41d21770 100644 --- a/ops/metrics/alloy/alloy-infra.alloy +++ b/ops/metrics/alloy/alloy-infra.alloy @@ -131,6 +131,18 @@ loki.process "scrub_credentials" { expression = "://[^:@/ ]+:([^@ ]+)@" replace = "***" } + + // ── Секреты в query-строке (#3354) ───────────────────────────────────────── + // То же выражение, что в alloy-apps.alloy, и по той же причине — подробности + // там. Здесь оно нужно не меньше: на инфра-хосте живут Forgejo и GlitchTip, + // у обоих есть ручки с `?token=` в адресе, и их логи идут в тот же Loki. + // + // Группа захвата — на ЗНАЧЕНИИ: Alloy заменяет содержимое групп, а не весь + // совпавший фрагмент (как у DSN выше, где группа на пароле). + stage.replace { + expression = "(?i)[?&][\\w.-]*(?:secret|token|api[-_]?key|apikey|access[-_]?token|password|signature|sig)=([^&\\s\"'<>]+)" + replace = "***" + } } loki.relabel "journal" {