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

Bugfix #23

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

Bugfix #23

wants to merge 1 commit into from

Conversation

Peronia
Copy link
Contributor

@Peronia Peronia commented Jul 22, 2020

Bugfix: don't use only the first entry for update
better errorhandling
use more functions

Bugfix: don't use only the first entry for update
better errorhandling
use more functions
Strubbl added a commit to Strubbl/Bash-INWX-DynDNS that referenced this pull request Oct 29, 2021
This commit fixes a race condition, where only the first inwx id of an
array of multiple ids gets updated. The section to save the old v4 and
v6 address is moved out of the for loop in order to have these variables
saved globally. This way we have still the old ip address saved, even in
the second run of the for loop.

This commit is a similar solution to the more complex solutions in gehaxelt#23
and gehaxelt#25
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