fix(security): replace hardcoded database credentials with env vars - #36
Conversation
miccy
commented
Aug 20, 2026
- Replace hardcoded credentials in docker/docker-compose.yml with environment variable interpolation
- Add documentation for database env vars in docker/README.md
- Add automated test in tests/docker-config.test.ts to verify docker-compose configuration
- Replace hardcoded credentials in docker/docker-compose.yml with environment variable interpolation - Add documentation for database env vars in docker/README.md - Add automated test in __tests__/docker-config.test.ts to verify docker-compose configuration
|
Important Approval pendingCodeRabbit has no unresolved comments, but it skipped the latest review. Use the checkbox below to review the latest commit. CodeRabbit will approve the changes if it finds no blocking issues.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 167840ca55
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…s-6384860232121466724 Signed-off-by: Miccy <support@miccy.dev>