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

Update nsupdate-dynip #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update nsupdate-dynip #1

wants to merge 1 commit into from

Conversation

preis98
Copy link

@preis98 preis98 commented Oct 16, 2024

I noticed that if I run the script in the crontab every 10min it updates the zone every time, even though the IP addresses are the same. I then sat down and saw that it apparently tries to compare two strings or a string and a number and then it thinks the IP is not the same and therefore executes an nsupdate. I fixed the whole thing by running the IP and OLDIP in a subshell and adding an xargs to both. I removed the awk and since then the script recognises that both IP addresses are the same.

I noticed that if I run the script in the crontab every 10min it updates the zone every time, even though the IP addresses are the same. I then sat down and saw that it apparently tries to compare two strings or a string and a number and then it thinks the IP is not the same and therefore executes an nsupdate. I fixed the whole thing by running the IP and OLDIP in a subshell and adding an xargs to both. I removed the awk and since then the script recognises that both IP addresses are the same.
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 this pull request may close these issues.

1 participant