Skip to content
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.

Sign up for GitHub

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

Jump to bottom

update for dns.la new version api #4830

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

update for dns.la new version api #4830

wants to merge 5 commits into from

Conversation

cnssl
Copy link
Contributor

@cnssl cnssl commented Oct 18, 2023 •
edited
Loading

  1. update for dns.la brandly new version api.
  2. correct bugs while parsing "domain_id" for previous edition

@github-actions
Copy link

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 ✨

@Neilpang
Copy link
Member

能解释一下吗

@cnssl
Copy link
Contributor Author

cnssl commented Oct 30, 2023

能解释一下吗

DNSLA系统升级，目前的脚本对接了新的API

@Neilpang
Copy link
Member

Neilpang commented Dec 3, 2023

之前的 id 和 key 的方式完全不能用了吗?

@cnssl
Copy link
Contributor Author

cnssl commented Dec 4, 2023 •
edited
Loading

之前的 id 和 key 的方式完全不能用了吗?

之前的旧版API的id和key是完全兼容的。只是新版API的请求路径都发生了改变，不更新版本，原来的脚本就不能使用了。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@cnssl @Neilpang