fix(infra): GlitchTip EMAIL_URL dummy:// → consolemail:// (crashes Django on start) #223
No reviewers
Labels
No labels
admin
analytics
auth
automation
bug
business
chore
ci
compliance
data
data-moat
docs
duplicate
dx
enhancement
Fable 5 ревью
feedback/max
generative
GG-форсайт
needs-discussion
needs-human
observability
pause-bots
performance
priority/p0
priority/p1
priority/p2
priority/p3
scope/backend
scope/db
scope/devops
scope/frontend
scope/qa
scrapers
security
site-finder
stage/1
stage/2
status/blocked
status/done
status/needs-analysis
status/needs-fix
status/qa
status/ready
status/review
status/wip
tech-debt
tradein
ux
week ревью 1
wontfix
вторичка
ИРД
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: lekss361/gendesign#223
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "fix/glitchtip-email-url-consolemail"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Контекст
После merge #205 и развёртывания GlitchTip 6.1.6 в проде, контейнер
glitchtip-webупал в restart-loop с ошибкой Django:Причина
django-environв новой версии GlitchTip валидирует схемуemail_url()на старте и НЕ знаетdummy://. Допустимые схемы:smtp:///smtps:///submission://— реальная отправкаconsolemail://— печатает в stdout, удобно для self-hosted без SMTPdummymail://— silent no-op (то что я изначально хотел указать в брифе)filemail:///memorymail:///locmemmail://Это была моя ошибка в брифе devops-агенту для PR #205.
Фикс
Одна строка в
docker-compose.prod.yml:consolemail://лучшеdummymail://потому что вdocker logsвидна попытка отправки email (например, при reset-password) — удобно дебажить. Шлёт никуда, безопасно.Hotfix на VPS
Та же замена была применена руками через
sed -i 's|EMAIL_URL: dummy://|EMAIL_URL: consolemail://|g' /opt/gendesign/docker-compose.prod.ymlчтобы разблокировать GlitchTip без ожидания этого PR. Ноgit reset --hardпри следующем deploy перетёр hotfix → этот PR делает фикс постоянным.Test plan
consolemail://стартует на VPS под hotfix (verified)glitchtip-webостаётся healthydocker exec glitchtip-web ./manage.py shell→from django.core.mail import send_mail; send_mail(...)печатает в stdoutRelated
Code review: APPROVE
Scope: 1 file, +1/-1 — hotfix для
glitchtip-webrestart-loop после merge #205.Verified
docker-compose.prod.ymlline ~141, env block сервисаglitchtip-web,EMAIL_URL: dummy://→EMAIL_URL: consolemail://. Никакие другие строки/сервисы не затронуты.consolemail://— валидная django-environ схема (resolves todjango.core.mail.backends.console.EmailBackend). Полный allowlist django-environ:smtp/smtps/submission/consolemail/dummymail/filemail/memorymail/locmemmail— bugInvalid email schema dummyустраняется.consolemail://overdummymail://обоснован: stdout-видимость попыток отправки вdocker logsпомогает дебажить (например, password-reset attempts), при этом email никуда реально не уходит → безопасно для self-hosted без SMTP.true, base SHA5a7d405совпадает с current main HEAD, no conflicts.feat(nspd): denorm) затрагиваетdata/sql/+backend/app/— no overlap сdocker-compose.prod.yml.Findings
Нет blocking issues. Nitpicks:
.claude/rules/deploy.mdили vaultfixes/glitchtip-email-url-schema.mdчтобы не наступить повторно при добавлении email-зависимых Django-сервисов (Sentry-like). Не блокирует merge.DEFAULT_FROM_EMAILкак opt-in для real SMTP — текущееerrors@gendsgn.ruостаётся, но приconsolemail://оно фактически no-op (only printed). Не баг, но можно в follow-up закомментировать с пометкой "activate when SMTP configured" для чистоты конфига.Test plan post-merge
docker compose -f docker-compose.prod.yml up -d glitchtip-webпосле deploy — контейнер healthy, нетImproperlyConfiguredв логахdocker logs glitchtip-web 2>&1 | grep -i "email schema"→ emptysend_mail()в Django shell → видно вdocker logsVerdict: APPROVE — fix корректный, минимальный, точно соответствует root cause.
Blocked for auto-merge per
.claude/rules/git-pr.md(touchesdocker-compose*.yml). Human merge required to make hotfix permanent (VPS already patched manually,git reset --hardследующего deploy перетрёт hotfix без этого merge).