-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add DNS provider for Octenium #2661
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
Conversation
Details |
|
Based on the log, the API documentation is wrong. I updated the implementation. |
Details |
|
So the documentation is very weak... I improved the parsing of the response, but I already set all the required fields. I added some debug logs, can you try again? |
|
Yes, I noticed that the docs need many improvments and fixes, and I contacted them about that. Details |
As you can see the query parameters are set:
And bonus:
The API is rate-limited, but there is no documentation about that... |
|
Maybe it's because there is a non-documented min TTL. I increased the default TTL. |
|
I found the doc about rate limits:
lego is doing 4 calls per domain, then 8 for 2 domains (base + wildcard). So I think, either you run lego several times or the doc is wrong again. Note that 20 req/s is very limited. |
|
After the last commit, the rate limit issue is resolved, but there is still "error: missing order-id." Waiting for them to fix it. Details |
This comment was marked as off-topic.
This comment was marked as off-topic.
|
I updated the implementation, can you try again? |
|
Now all requests are ok, but another error occurred Details |
|
Adding |
dmke
left a comment
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.
LGTM
go mod tidyCloses #2660
Ping @abdeldjalil-fellah, can you run the command (with your domain, email, credentials, etc.)?
How to test this PR?
git clone https://github.com/ldez/lego.git cd lego git checkout feat/dns/octeniummake:make buildmake:go build -o dist/lego ./cmd/lego