acme.sh icon indicating copy to clipboard operation
acme.sh copied to clipboard
verified publisher icon acmesh-official

Replace Yandex PDD api with Yandex360 api

Open ProKn1fe opened this issue 2 years ago • 6 comments

Old Yandex PDD api was deprecated and replaced with Yandex360 api (more info #4555).

ProKn1fe avatar Apr 16 '23 18:04 ProKn1fe

Welcome Please make sure you're 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 ✨

github-actions[bot] avatar Apr 16 '23 18:04 github-actions[bot]

Welcome Please make sure you're 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 ✨

I have only one question i should update existing yandex pdd wiki item or create new? It will broke sequence because it must be removed due to complete inaccessible.

ProKn1fe avatar Apr 16 '23 18:04 ProKn1fe

Waiting for this merge

DrySock avatar May 23 '23 12:05 DrySock

So I renamed the old one to old and put the new 360 script in /dnsapi but still get an error though: Can not find dns api hook for: dns_yandex You need to add the txt record manually. Add the following TXT record: blah-blah acme.sh --renew --dns dns_yandex -d '*.vadim.com.ru' -d vadim.com.ru --dnssleep3600 and the dnssleep does not work for the countdown either

wolf4914 avatar May 23 '23 18:05 wolf4914

fix the dns test first: https://github.com/acmesh-official/acme.sh/wiki/DNS-API-Test

Neilpang avatar May 26 '23 21:05 Neilpang

So I renamed the old one to old and put the new 360 script in /dnsapi but still get an error though: Can not find dns api hook for: dns_yandex You need to add the txt record manually. Add the following TXT record: blah-blah acme.sh --renew --dns dns_yandex -d '*.vadim.com.ru' -d vadim.com.ru --dnssleep3600 and the dnssleep does not work for the countdown either

  1. Yandex DNS API name has been changed. The new value for the --dns argument must now be set as dns_yandex360.
  2. You must issue a new token for the updated Yandex API. Please refer to this guide for more information: https://yandex.ru/dev/api360/doc/concepts/access.html

If you have any questions, please refrain from asking them in the pull request body. Instead, kindly refer to the corresponding issue https://github.com/acmesh-official/acme.sh/issues/4555 for further assistance

WhiteAls avatar May 26 '23 22:05 WhiteAls

fix the dns test first: https://github.com/acmesh-official/acme.sh/wiki/DNS-API-Test

What should be fixed? I don't see yandex pdd tests in current DNS.yml

ProKn1fe avatar Dec 29 '23 08:12 ProKn1fe

What should be fixed? I don't see yandex pdd tests in current DNS.yml

You need to run these tests on your own in your forked repository. Just follow all the instructions provided in the attached link. Also, did you read this? https://github.com/acmesh-official/acme.sh/wiki/DNS-API-Dev-Guide

WhiteAls avatar Dec 31 '23 21:12 WhiteAls

What should be fixed? I don't see yandex pdd tests in current DNS.yml

You need to run these tests on your own in your forked repository. Just follow all the instructions provided in the attached link. Also, did you read this? https://github.com/acmesh-official/acme.sh/wiki/DNS-API-Dev-Guide

And what if i no more have domain with this api to test?

ProKn1fe avatar Jan 15 '24 11:01 ProKn1fe

I have no more domains in yandex360, so i can't test it.

ProKn1fe avatar Jan 27 '24 13:01 ProKn1fe

Hello. I would like to resume this topic. Tell me what needs to be done to make a merge?

dimmik80 avatar Jun 17 '24 10:06 dimmik80