fire1ce / UDM-Cloudflare-DDNS

Cloudflare DDNS container for UDM
https://3os.org
14 stars 2 forks source link

Failed to perform DNS update with multiple zone ID #3

Open PeterWan816 opened 1 month ago

PeterWan816 commented 1 month ago

The API have access to all zones, but with different subdomains. (6 sub domains which at this point points to the same ip) that needs to be updated by the same DDNS setup. despite of the Log showing that the relevant subdomain has been updated, it did not update according to the Cloudflare Dashboard. the settings was separated like below:"sub1.example.com, sub2.example.com, sub3.example.com,..., sub6.example.com"

PeterWan816 commented 1 month ago

the bug has been found, its near the Telegram notification line. Author you copy and pasted too fast, exit 0 ends the entire program, the program should not use exit 0 at the section, instead it should simply continue with no further action needed. I solved the issue by commenting the exit 0 line and added echo "" incase the condition is useful in the future.