-
-
Notifications
You must be signed in to change notification settings - Fork 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
pdns: optional custom API version #2019
Conversation
Do we agree that I close the issue and flag it as "question" and you still created a PR? |
Hi,
As described in the PR, PowerDNS has root API endpoints to provide the version information (https://doc.powerdns.com/recursor/common/api/endpoint-api.html). Some PowerDNS deployment don't have these endpoints exposed and without them it is not possible to auto-detect PowerDNS API version and the whole process with lego fails. Using custom variable to set the API version, you can skip the API version auto-detection and the process will continue.
I found I am not the only one with this issue, but it's up to you to see whether you want to have this feature or not.
Best Regards,Marijan
|
I agree, powerdns 4.* does not have this endpoint, so i had to hack into the code in order to set the apiVersion to 1. I am setting PDNS_API_URL=https://server/api/v1 - all works until the PATCH command. The patch command is using zone.URL as path and it is coming from pdns |
f04c8df
to
7b2b7e6
Compare
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.
Even if I disagree with the fact of opening a PR when an issue was closed, I will merge this PR anyway.
I hope this will not happen again and discussion will happen inside the issue BEFORE opening a PR.
Some PowerDNS servers don't have exposed the root API endpoints and PowerDNS API version auto-detection fails. Use the optional parameter
PDNS_CUSTOM_API_VERSION
to set the version number in such situations.related to #2016