Убирает DB-секреты из docker-compose.yml и COPY backend/.env из Dockerfile. Compose передаёт production-конфигурацию через backend/.env; добавлены инструкции и bump backend 0.6.7.
Убирает DB-секреты из docker-compose.yml и COPY backend/.env из Dockerfile. Compose передаёт production-конфигурацию через backend/.env; добавлены инструкции и bump backend 0.6.7.
Блокирующее замечание: в репозитории отсутствует .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 работают.
Повторный полный 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 main2026-08-19 21:22:11 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Убирает DB-секреты из docker-compose.yml и COPY backend/.env из Dockerfile. Compose передаёт production-конфигурацию через backend/.env; добавлены инструкции и bump backend 0.6.7.
Блокирующее замечание: в репозитории отсутствует
.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 работают.Повторный полный 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.