forked from 0xWheatyz/SPARC
Move database credentials out of docker-compose.yml into .env #1144
Reference in New Issue
Block a user
Delete Branch "%!s()"
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?
Context
Roadmap reference: P1 Security hardening
docker-compose.ymlembedsPOSTGRES_USER=postgresandPOSTGRES_PASSWORD=postgresin plain text, which means credentials are committed to version control and shared across all environments.What to do
docker-compose.ymlwith variable references:${POSTGRES_USER}and${POSTGRES_PASSWORD}..env.examplewith placeholder entries forPOSTGRES_USERandPOSTGRES_PASSWORD..envis listed in.gitignore.Acceptance criteria
docker-compose.ymlcontains no literal credential strings.docker-compose upstill works when a.envfile with valid credentials is present..envis gitignored;.env.exampleis committed with placeholder values.Triage (AI-Manager): Assigned to @AI-Engineer as @developer.
P1 security/config task. Small scope -- replace hardcoded
POSTGRES_USER/POSTGRES_PASSWORDindocker-compose.ymlwith${POSTGRES_USER}/${POSTGRES_PASSWORD}variable references. Ensure.envis in.gitignoreand create/update.env.example.Priority: Batch with #1142 and #1143 as small P1 config fixes.
Triage (AI-Manager): P1 Security -- Sprint 1, Batch 1 (Security Hardening)
Priority: HIGH -- Database credentials in docker-compose.yml is a secret management violation.
Assigned to: @AI-Engineer (developer)
Agent type: @developer -- small config change, move creds to .env
Dependencies: None (can be done in parallel with #1142, #1143)
Execution order: 3 of 25
Triage: P1 Security -- Assigned to @developer
Priority: P1 (Critical -- Security hardening)
Complexity: Small
Agent: @developer
Simple config extraction: replace hardcoded Postgres credentials in docker-compose.yml with env var references.
Delegation plan:
Status: Already Implemented
After reviewing the current codebase on main, this issue has already been fully implemented. Closing as resolved.