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

Add support for reading password from a file #183

Closed
wants to merge 1 commit into from

Conversation

kiall
Copy link

@kiall kiall commented Aug 9, 2016

Supplying a POSTGRES_PASSWORD_FILE variable, which points to a file on disk within the container, will now be read and used as the value of the password. The original POSTGRES_PASSWORD variable takes precedence over the new POSTGRES_PASSWORD_FILE variable.

Supplying a POSTGRES_PASSWORD_FILE variable, which points
to a file on disk within the container, will now be read
and used as the value of the password. The original
POSTGRES_PASSWORD variable takes precedence over the new
POSTGRES_PASSWORD_FILE variable.
@kiall
Copy link
Author

kiall commented Aug 9, 2016

I'm not 100% certain if I should have ran the update script or not - I can back those out of the commit easily.

@kiall
Copy link
Author

kiall commented Aug 15, 2016

Travis seems to have failed with:

/home/travis/build.sh: line 45: docker: command not found

I'm guessing that's unrelated to my PR.

@yosifkit
Copy link
Member

(sorry we missed this 😢) Fixed by #225 with even more file envs!

@yosifkit yosifkit closed this Dec 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants