-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Using dokku-acl with dokku-letsencrypt breaks auto-renewal #22
Comments
Sorry for getting to this so late:
|
Thanks for your reply! I did the following:
|
Where is the crontab file specified? |
I edited the crontab w ith
|
The default username for dokku is If we did, that wouldn't break anything in dokku core, and the acl plugin would continue working as expected (the logged in Anyone have any thoughts on this? |
I think that makes sense, I’d appreciate the rename of the default Dokku’s username to I found this issue a few days ago when I was investigating the difficulties of allowing my friend to host his website on my server. I spent some time testing if the reported issue is still a problem nowadays, and came up to the conclusion that as long as your SSH key is called But definitely renaming the user from |
This not only makes running commands manually much simpler, but removes some confusion around "dokku" / "default" user as the superuser (some places need both). Fixes dokku-community#22 Fixes dokku-community#36
We use
dokku-acl
together with thedokku-letsencrypt
plugin. For auto-renewal of certificates, the plugin creates a cronjob for the dokku-User:which in turn executes
which fails on our hosts:
What would be the best way to fix this issue? I thought about some ways, but I didn't find one which works well...
export DOKKU_SUPER_USER=dokku
. So if the cronjob would exportNAME
it should work, right?letsencrypt:auto-renew
toDOKKU_ACL_USER_COMMANDS
. But sinceletsencrypt:auto-renew
callsletsencrypt
andcerts:add
, we would need to whitelist those functions for all apps as well, which is undesirable.The text was updated successfully, but these errors were encountered: