-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Added new 'dns' provider script for https://dns.services #4151
Added new 'dns' provider script for https://dns.services #4151
Conversation
I have made a few simple certificate requests, which worked fine for me. NB: I am co-owner of DNS.services |
During the testing period in the "DNS API Test" page I found that it failed with the shfmt and shellcheck - I've fixed these so it is POSIX compliant. I found that the rundocker.sh script has an error in line 411 due to "==" vs "=" usage causing issues for me which diff --git a/rundocker.sh b/rundocker.sh
index 58e14c7a9..b798be827 100755
--- a/rundocker.sh
+++ b/rundocker.sh
@@ -411,7 +411,7 @@ testplat() {
if [ "$_rret" != "0" ] ; then
_info "Failed: $plat"
_FAILED_PLATS="$_FAILED_PLATS$plat "
- if [ "$TRAVIS" = "true" ] || [ "$GITHUB_ACTIONS" == "true" ]; then
+ if [ "$TRAVIS" = "true" ] || [ "$GITHUB_ACTIONS" = "true" ]; then
return "$_rret"
fi
fi The
Anyhows, now that shellcheck, shfmt and the build actions for the DNS.yml all pass then the PR is ready for merge. |
please check this guide: |
I tried updating the wiki but get a rendering error and I've tried to revert my change, but same error. So I've added it again as https://dillinger.io/ renders the full wiki markdown page just fine and I hope it is GitHub's backend that isn't rendering. Perhaps it has just grown to big over time and needs to be split up? |
No, just copy and add your segment to the page. |
I tried - I think it is something to do with the backticks because when I used 4 spaces to the code blocks for export and examples then it renders just fine. It has been added now. |
@Neilpang is there anything I'm missing here for the pull request to be complete?
|
I just added more tests. please clieck the |
The "docker" action stage fails as there seems to be something wrong with Acme staging, or how it revokes certificates - it issues certs just fine but the script fails with
Is this normal @Neilpang ? |
@Neilpang I've tried again... it failed again. It looks like I have to wait for acmetest to be fixed https://github.com/acmesh-official/acmetest as it is failing on all docker image actions at the moment. |
@Neilpang I've tried again.... the actions for I'm gonna try to test the This is where it fails using
After that it continues with the wildcard certificate test using the dns plugin and that works just fine and issues certificates using the dns plugin (dns_dnsservices). But when it revokes the issued certificate if fails even though the acme service responds with HTTP 200
|
please rerun the tests again. |
@Neilpang I've tried re-running the actions - it still fails. I'm at home now and I'm testing using TEST_LOCAL=1 on It is quit difficult to debug...
They are all different errors happening at different stages/times - this is both happening in github action(s) when I re-run or update the pull request with a new commit like earlier today but also in my local acmetest using TEST_LOCAL=1. Do you have any advice on how to do a proper test? Eg with a mock of the ACME service so I can disregard these unless you think the HTTP2/ 400 response is cause from the letest.sh script it self because of lost variables like the _add function it looses (not the value of the ${api} variable, but the I've changed my domain and plugin to "****" like the output from the github actions Failure due to "HTTP/2 400"
Failure due to missing "_add" function where it worked in the run before:
Failure due to missing "_add" function in the assertcmd for "acmetestXyzRandomName.$TestingDomain"I've 'ed out most of the acme.sh help it writes when run with incomplete commands, as if the ${api} value has been reset/lost at some point.
|
sorry, there was a bug in acmetest. I just fixed acmesh-official/acmetest@9c4e5cb please try again. |
…that was explicitly disallowed as it is not _acme-challenge
@Neilpang all tests pass in my branch now :-) |
please rebase code. Openbsd test is added. |
I would love to - but github crashes with "Ooops!!! 500". |
I think Github has some issues with Actions tonight. Will look at it tomorrow as I can't access my branch I created the pull-request from (still get "Ooops 500"). Git failed on a few OS'es and the reset was cancled.
|
@Neilpang I think something went wrong with my merge yesterday as I'm still getting Ooops 500 on my branch as it seems to have a melt down.
This is git operations beyond my skill level. Ignore this: created a new branch and prepped it, and then actions started working on this branch and actions are running. Don't see OpenBSD in the actions flow though... Ignore the ignore above: my branch is f...ed. Tried to trigger workflow, but ... no luck.
I think I should create a new fork and branch and pull request, ok @Neilpang ? |
@Neilpang I've re-tried to to build but have found tha tthe "acmeTestTxtRecord" that is created isn't deleted in the acmetest testing, so I have 30+ of these Most of the tests are going through now. Solaris is failing (looping) with the following - thousands of times:
Ubuntu (LetsEncrypt.org_test, (STAGING) Pretend Pear X1) is failing with "wait for connection to establish..." and not getting anywhere (as of this writing it is through 522 of these). Ubuntu (ZeroSSL.com, ZeroSSL ECC Domain Secure Site CA, ZeroSSL RSA Domain Secure Site CA, github... seems to hang though the tests pass
Windows (LetsEncrypt.org_test, (STAGING) Pretend Pear X1) is stuck "Run acmetest" like above Ubuntu ZeroSSL test:
This is the last test I'm waiting for - I don't know what it is waiting for... |
that means something wrong. you need to fix your code. |
If there are some of the tests failed. please just click the re-run button to run the failed tests again, untill all them them are passing. |
@Neilpang all tests pass now after fixing for OpenBSD (which works the same as NetBSD). |
Thank you so much. here we go |
1 similar comment
Thank you so much. here we go |
This dns provider script is for the Danish DNS hosting provider DNS.Services (https://dns.services).
The script have been developed on Unbuntu 20.04 and tested for single domain, multiple domain and wildcard certificate issuing in --test, --staging and production.
Usage example