Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggestion: Use .env for development environment variables instead of docker-compose.yml #18

Closed
JakeChampion opened this issue Mar 9, 2016 · 1 comment
Labels
maintenance Technical tasks that might make things better

Comments

@JakeChampion
Copy link
Contributor

Currently the README advocates storing confidential environment variables inside docker-compose.yml for local development. This becomes a bit messy because that file is included within git but will become personal to each developer's set-up (API keys, auth tokens, database names etc). A developer may accidentally commit their environment variables into the git history (which is a bad thing tm).

Have you thought about using dotenv instead of placing the environment variables within docker-compose.yml?

docker-compose.yml is commited into the repo whereas .env can be ignored in .gitignore. This means confidential env vars can be stored in it safely and because the file is ignored by git it won't be deployed to production.

@onishiweb
Copy link
Contributor

👍

@rowanmanning rowanmanning added maintenance Technical tasks that might make things better priority: low labels Mar 24, 2016
rowanmanning added a commit that referenced this issue Apr 7, 2016
This prevents developers from accidentally committing sensitive
information to the repo. Addresses #18.
rowanmanning added a commit that referenced this issue Apr 7, 2016
This prevents developers from accidentally committing sensitive
information to the repo. Addresses #18.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Technical tasks that might make things better
Projects
None yet
Development

No branches or pull requests

3 participants