-
Notifications
You must be signed in to change notification settings - Fork 60
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 simple way to update Drupal's admin password #193
Conversation
Will be adding another step. During the Tech call it was requested to pipe the password into the secrets file. I'll add this to this Pull request. |
OK, ready for testing. Super simple low hanging fruit |
Running this off a
I dealt with this in the |
Ok, this tests out fully for me. PWD is set in Drupal, I can log in, and the contents of that secret file is correct after the command runs. I'm happy to merge this unless someone else wants to test it first... |
Thanks for reviewing |
@Islandora-Devops/committers are we good with merging this? |
6c38fa0
to
f364b7e
Compare
@DonRichards, please resolve conflicts. |
@seth-shaw-unlv Not sure why it thought there was a conflict but either way it's resolved now. |
It worked to change the Drupal password but failed to update the live secrets file:
Are we expected to run the make command as root? |
OH!!! That's what you meant. No that isn't expected. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
@seth-shaw-unlv Got it!! Man that took too long. I added a check that basically looks to see if the Makefile/USER can write to the password file. If not it tries running the echo $password with sudo, otherwise it just echos the password to the file. This seems like the most reasonable approach. |
Oh, great! Thanks |
To make it easier to reset admin password this make command keeps it simple.
To test use the readme instructions at the bottom of the "Secrets" section.
I tested this on mac and linux so it should work for either.