-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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 ZoneEdit certbot plugin #4208
base: develop
Are you sure you want to change the base?
Add ZoneEdit certbot plugin #4208
Conversation
9551d9d
to
1e32280
Compare
Thank you! I closed #4188 |
Thanks, but I just need someone to confirm that this works as expected using the test docker image above. |
Just tried to initiate a DNS challenge with zoneedit, received the following error:
|
Hey, thanks for testing. I think I fixed the error could you please try again? |
Docker Image for build 4 is available on Note: ensure you backup your NPM instance before testing this image! Especially if there are database changes |
Fixes #4171 without the whitespace issues from #4188.
Credits to #4188 - I hadn’t noticed this PR when I initially added ZoneEdit. Given that it has been a week since the whitespace issues, I decided to move forward and publish my PR to help maintain a faster development cycle.