Skip to content

Implement DNS API for Efficientip SOLIDserver #6333

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

Open
wants to merge 25 commits into
base: dev
Choose a base branch
from

Conversation

EfficientIP-Labs
Copy link

Implement DNS API for EfficientIP SOLIDserver DNS providers.

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
Author

Waiting for feedback. Thanks in advance.

@alexissavin
Copy link

@Neilpang You seem to be one of the rare active maintainer currently. Any chance to see this pull request accepted and merged ?
Many thanks for your feedback.
Kind regards.

@Neilpang
Copy link
Member

Neilpang commented May 7, 2025

did you read this comment?

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, 都必须确保通过这个测试.

@alexissavin
Copy link

Hello @Neilpang,

I have read and followed the instructions (even closed initial PR created by mistake to the master branch).
I have followed the API development guide and complied with the documented testing requirements.
I also have updated the documentation as indicated.

I did answered the comment you refer to, but I'm not sure what answer you expect.

Many thanks for your feedback.

kind regards.

@Neilpang
Copy link
Member

Neilpang commented May 8, 2025

@alexissavin
Copy link

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

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

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.

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.

3 participants