Спрятать production-секреты Docker в env #29

agent · 2026-08-19T21:19:13Z

agent commented

2026-08-19 21:19:13 +00:00

Убирает DB-секреты из docker-compose.yml и COPY backend/.env из Dockerfile. Compose передаёт production-конфигурацию через backend/.env; добавлены инструкции и bump backend 0.6.7.

agent added 1 commit 2026-08-19 21:19:13 +00:00

fix: inject production secrets via env file bb3789c01c

agent commented

2026-08-19 21:20:24 +00:00

Блокирующее замечание: в репозитории отсутствует .dockerignore, а Dockerfile.backend:6 делает COPY backend ./backend. Поэтому локальный backend/.env попадает в build context и копируется в промежуточные deps/build-слои (и может остаться в build cache), несмотря на удалённый COPY backend/.env из финального runner-слоя. Добавьте корневой .dockerignore с backend/.env (лучше исключить все env-файлы, оставив *.env.example) и проверьте сборку. docker compose config с env_file проходит; Git-ignore и runtime env injection работают.

Блокирующее замечание: в репозитории отсутствует `.dockerignore`, а `Dockerfile.backend:6` делает `COPY backend ./backend`. Поэтому локальный `backend/.env` попадает в build context и копируется в промежуточные `deps/build`-слои (и может остаться в build cache), несмотря на удалённый `COPY backend/.env` из финального runner-слоя. Добавьте корневой `.dockerignore` с `backend/.env` (лучше исключить все env-файлы, оставив `*.env.example`) и проверьте сборку. `docker compose config` с `env_file` проходит; Git-ignore и runtime env injection работают.

agent added 1 commit 2026-08-19 21:21:06 +00:00

fix: exclude env files from Docker context 1142a9755a

agent commented

2026-08-19 21:21:35 +00:00

Повторный полный review PR #29 после 1142a97: предыдущий блокирующий риск устранён — корневой .dockerignore исключает **/.env/**/.env.*, поэтому backend/.env больше не попадает в Docker build context; финальный runner также не копирует env-файл. docker compose config --quiet проходит, env_file подключён, версия backend 0.6.8 согласована с CHANGELOG. Блокирующих замечаний не осталось, approve.

Повторный полный review PR #29 после `1142a97`: предыдущий блокирующий риск устранён — корневой `.dockerignore` исключает `**/.env`/`**/.env.*`, поэтому `backend/.env` больше не попадает в Docker build context; финальный runner также не копирует env-файл. `docker compose config --quiet` проходит, `env_file` подключён, версия backend 0.6.8 согласована с CHANGELOG. Блокирующих замечаний не осталось, approve.

admin merged commit a090ce37ad into main

2026-08-19 21:22:11 +00:00

admin referenced this issue from a commit

2026-08-19 21:22:13 +00:00

Merge pull request 'Спрятать production-секреты Docker в env' (#29) from fix/production-env-secrets into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: admin/family_budget#29