-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Support database passwords in db_spec for postgres databases #695
Conversation
…d point to it with the PGPASSFILE environment variable.
I have successfuly ran this patch against a Drupal7 CLI site-install with the Docker testbot:
Will test Drupal8 next |
With this patch Drupal8 site-install and test also works with the testbot:
|
@weitzman @greg-1-anderson feel free to proceed with the pull when possible, please |
Support database passwords in db_spec for postgres databases
Merged to master. It would probably be innocuous to add to the 6.x branch as well, if desired, as behavior does not change unless you define a 'password' item in your db spec for postgres. |
@greg-1-anderson have a look: http://testbot.drupal-pt.org/ Drupal8 Testbot for PostgreSQL |
I haz a happy. |
If 'password' is set in a pgsql db_spec, then write a pgpass file, and point to it with the PGPASSFILE environment variable.
This should work for things like sql-cli and site-install. More work necessary for sql-dump and sql-sync.