Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@EfficientIP-Labs
Copy link
Contributor

Implement DNS API for EfficientIP SOLIDserver DNS providers.

@github-actions
Copy link

Welcome
First thing: don't send PR to the master branch, please send to the dev branch instead.
Please make sure you've read our DNS API Dev Guide and DNS-API-Test.
Then reply on this message, otherwise, your code will not be reviewed or merged.
Please also make sure to add/update the usage here: https://github.com/acmesh-official/acme.sh/wiki/dnsapi2
We look forward to reviewing your Pull request shortly ✨
注意: 必须通过了 DNS-API-Test 才会被 review. 无论是修改, 还是新加的 dns api, 都必须确保通过这个测试.

@EfficientIP-Labs
Copy link
Contributor Author

Waiting for feedback. Thanks in advance.

@alexissavin
Copy link
Contributor

alexissavin commented May 9, 2025

Well I'm surprised, the error in the run you refer is not with the code associated with the PR.

I have re-sync my branch with the official repo. All checks are passing: https://github.com/EfficientIP-Labs/acme.sh/actions/runs/14923396036/job/41922851791

@neilpang, would you be kind to give it a new try ?

Kind regards.

export _H2="Authorization: Basic ${EfficientIP_CredsEncoded}"
else
TS=$(date +%s)
Sig=$(printf "%b\n$TS\nPOST\n$baseurlnObject" "${EfficientIP_Token_Secret}" | openssl dgst -sha3-256 | cut -d '=' -f 2 | tr -d ' ')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you not use the openssl here?
you can use the functin _digest

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well I can indeed, but I would then need to modify the following line in the acme.sh file to include the sha3-256 digest:

_digest() {
...
if [ "$alg" = "sha3-256" ] || [ "$alg" = "sha256" ] || [ "$alg" = "sha1" ] || [ "$alg" = "md5" ]; then
...

Would that be ok ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes

@alexissavin
Copy link
Contributor

Proposed changes address your request. All DNS-API-Test are passing in my branch (https://github.com/EfficientIP-Labs/acme.sh/actions/runs/15239890351).
Not sure why the "Ubuntu (ZeroSSL)" check is failing.
This failure does not seems related to the proposed code.

@neilpang
Copy link
Member

it seems that zerossl is down: https://status.zerossl.com/

@alexissavin
Copy link
Contributor

@neilpang is there anything else I can do to move forward with this pull request ?
I feel a bit stuck at the moment as I do not foresee any additional change to commit.

@EfficientIP-Labs
Copy link
Contributor Author

@neilpang all tests have finally passed: https://github.com/EfficientIP-Labs/acme.sh/actions/runs/15588147930

If you have the opportunity, can you trigger the pending workflows ?

Kind regards

export _H2="Authorization: Basic $EfficientIP_CredsEncoded"
else
TS=$(date +%s)
Sig=$(printf "%b\n$TS\nDELETE\n${baseurlnObject}" "${EfficientIP_Token_Secret}" | openssl dgst -sha3-256 | cut -d '=' -f 2 | tr -d ' ')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

openssl dgst

@alexissavin
Copy link
Contributor

Sincere apologies, I don't know how I missed it...
Fixed

@achauvinhameau
Copy link

interested in this one for proxmox @neilpang . Thanks in advance

@EfficientIP-Labs
Copy link
Contributor Author

@neilpang could you please re-run the check that fails for external reasons (Seems like some kind of rate limiting from cloudflare) ? Seeing this code integrated natively in your project would make life easier for many of my contacts. Thanks in advance.

@neilpang
Copy link
Member

neilpang commented Sep 7, 2025

I can not rerun it. please make a new push

@@ -0,0 +1,139 @@
#!/bin/sh
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use #!/usr/bin/env sh

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noted, thanks

@neilpang
Copy link
Member

neilpang commented Nov 9, 2025

your latest run is not passing:

https://github.com/EfficientIP-Labs/acme.sh/actions/runs/18615493475

@alexissavin
Copy link
Contributor

@neilpang - Finally got all tests to pass: https://github.com/EfficientIP-Labs/acme.sh/actions/runs/19298599984

It is quite confusing that this test suite very often fails for reasons completely external to the script itself.

I hope this will permit to move forward.

Kind regards

@alexissavin
Copy link
Contributor

Hello @neilpang,
Any chance you can approve the pending workflows ?
As all tests are finally passing, I hope you can merge this one.
Many thanks in advance.
Regards.

@neilpang neilpang merged commit b7fe7a4 into acmesh-official:dev Dec 8, 2025
40 of 42 checks passed
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

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.