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.

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

The script fails when the API replies nochg (ie, it got the same IP) #61

Open
marco-brandizi opened this issue Sep 4, 2024 · 0 comments · May be fixed by #62
Open

The script fails when the API replies nochg (ie, it got the same IP) #61

marco-brandizi opened this issue Sep 4, 2024 · 0 comments · May be fixed by #62

Comments

@marco-brandizi
Copy link

marco-brandizi commented Sep 4, 2024

As per subject, if you call the script from the same IP that YDNS has registered in its DNS server, the scripts fail, due to the fact that the API output has the format: nochg <IP> and update_ip_address() doesn't parse such output correctly.

Note that this isn't the same case where the script compares the current IP with the value it previously saved on a /tmp/ file. Rather, I'm on the same host (and same IP), but after a reboot, ie, the IP saved on /tmp/* is gone, but the API is queried with that same IP.

I've a fix in my fork and I'll send a PR.

@marco-brandizi marco-brandizi linked a pull request Sep 4, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant