fix: /debug убран (утечка секретов)
This commit is contained in:
@@ -0,0 +1,25 @@
|
||||
# Code Review v3 — Результаты (Соннет, 2026-08-09)
|
||||
|
||||
## 1. Структура — ✅ ОК
|
||||
package main по файлам, ~400 строк — дробить рано.
|
||||
|
||||
## 2. embed — ✅ ОК
|
||||
Но `tmpl, _ := ReadFile` — ошибка игнорируется.
|
||||
|
||||
## 3. proxyHandler — 🟡 мёртвый параметр
|
||||
`?bucket=` в URL не читается proxyHandler — можно убрать из downloadLink.
|
||||
|
||||
## 4. listVersions — 🟡 нет таймаута
|
||||
`context.Background()` без дедлайна.
|
||||
|
||||
## 5. defer в if — ✅ ОК
|
||||
Корректно, defer привязан к функции.
|
||||
|
||||
## 6. /debug — 🔴 КРИТИЧНО
|
||||
Секреты S3 отдаются публично. Убрать.
|
||||
|
||||
## 7. Dockerfile — ✅ ОК
|
||||
Паттерн нормальный. Рекомендация: добавить USER.
|
||||
|
||||
## 8. Итог
|
||||
Единственный блокер: /debug с утечкой секретов. Остальное — желательно.
|
||||
Reference in New Issue
Block a user