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

Deprecated parameter names #11

Open
mcfisch opened this issue Jul 20, 2021 · 2 comments
Open

Deprecated parameter names #11

mcfisch opened this issue Jul 20, 2021 · 2 comments

Comments

@mcfisch
Copy link

mcfisch commented Jul 20, 2021

When I try using the plugin as it is included with the linuxserver/swag docker container I'm getting a warning message saying the plugin name format is considered 'legacy' now:

root@47ec7d874239:/# certbot certonly --authenticator certbot-dns-cpanel:cpanel --certbot-dns-cpanel:cpanel-credentials
/config/dns-conf/cpanel.ini -d 'my-domain.com' -d '*.my-domain.com' -v
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Plugin legacy name certbot-dns-cpanel:cpanel may be removed in a future version. Please use cpanel instead.
Plugins selected: Authenticator certbot-dns-cpanel:cpanel, Installer None
Requesting a certificate for my-domain.com and *.my-domain.com
Performing the following challenges:
...

While debugging the plugin for a different issue I've also stumbled upon this and was able to get around it by using the following command instead with respective changes to the config as well.

Command:

certbot certonly --authenticator cpanel --cpanel-credentials /config/dns-conf/cpanel.ini -d 'my-domain.com' -d '*.my-domain.com'

Config:

cpanel_url = https://my-domain.com:2083
cpanel_username = my_user
cpanel_password = my_pw

However, this for now is more of a cosmetic problem than an actual issue - but who knows when the old format is going to get removed from the bot?...

@badjware
Copy link
Owner

badjware commented Aug 13, 2021

Thank you for bringing this to my attention; I like the new style a lot more. It's much more readable.

It looks like there should be no code change required, I will update the readme when I will have the time to test this properly.

@sevaa
Copy link

sevaa commented Jan 5, 2023

As of Certbot 2.x, the legacy plugin name is not supported.

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

No branches or pull requests

3 participants