Updates dns_cyon txt_body and adds validation

[InteractiveNinja]

Hello!

I recently attempted to update my domains certificate via acme.sh, but encountered issues with the txt validation.
After some debugging, I found at that the form body had changed a little.

Changes

zone ➡️ name
value ➡️ dnscontent

I also added a validation for the response being returned after the update, in hopes that it will actually bail if the update fails.
I didn't try it, but I think it should work.

Thanks for this great tool, and also thanks for maintaining support for cyon domains 👍

[github-actions]

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.
We look forward to reviewing your Pull request shortly ✨
注意: 必须通过了 DNS-API-Test 才会被 review. 无论是修改, 还是新加的 dns api, 都必须确保通过这个测试.

[mikch06]

That's the issue!
Thx a lot!

[dedors]

Waiting for this too, hope will get merged soon.
In the meantime, it's possible to add the TXT record manually.

[mikch06]

Waiting for this too, hope will get merged soon. In the meantime, it's possible to add the TXT record manually.

You can change the dns_cyon.sh plugin on your own side until the fix is merged:
/root/.acme.sh/dnsapi/dns_cyon.sh

worked fine for me.

[mike4054]

I'm waiting for that update too but I see this is closed now. Any chance that this will make it to the release somewhen?