-
-
Notifications
You must be signed in to change notification settings - Fork 205
Aliyun API without SDK dependency #253
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
Conversation
8a14828
to
1195118
Compare
Hi @qdm12 I change my ip but I haven’t found the record value been changed, I will check the code tomorrow(GTM+8) |
Thanks @cnjack feel free to also shares your logs tomorrow, to see if it fails anywhere. It should still operate the same as in list records then update the record. |
Hey @cnjack have you had time to try it again? Thanks!! |
sorry, I am out for a few days and came back yesterday, I will check it. |
got error:
|
|
Awesome @cnjack !! 💯 Thanks! I pushed it on this branch, feel free to try the image |
hi @qdm12 , you haven't pushed the docker image, https://github.com/qdm12/ddns-updater/runs/4003765333?check_suite_focus=true |
I got the same error( |
Oops sorry I forgot the CI was not automatically building images on branches. I pushed it manually, can you try pulling and run it again ( |
I think it is work in my machine, but maybe need check the IP is equal to the record's value before update recode
|
my fault, I set a record in my router's host file, so app get a LAN IP(not equal to WALN IP) and hit the update logic. ignore the above comment |
I think you can merge this PR |
Are you sure? EDIT: Also thanks for being so quick ⚡ |
you will hit the |
the key point is that I set a static ip in my router's host file, the app get a LAN ip(ip in host file) |
Ok that make sense. By the way, I received some Aliyun credentials from you by email, do you want me to try anything? I'll merge this soon, thanks again for your testing and also figuring out how to fix the signature thing (saved me some headaches!) |
yes, I sent the credential by email, you mentioned that you want setup the record if the record not existed, it will be more easier if you have a real credential. |
- Remove `region` parameter - Support AAAA records - Files split around
Great job! |
Hello @cnjack sorry for reviving this old PR, I'm trying to create a record for Debug logs:
|
Merged it, it now creates the record if it doesn't exist, although it won't create the domain. I think the domain should exist anyway if the user wants to update records on it. |
region
parameter