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.

Sign up for GitHub

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

Jump to bottom

Fix issue with detection whether to use ipv6 or ipv4 #498

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix issue with detection whether to use ipv6 or ipv4 #498

wants to merge 2 commits into from

Commits on Sep 27, 2024

  1. Provider ipv64.net: Fix issue with detection whether to use IPv6 or IPv4 

    The issue was occurring while checking whether the provider name contained
"ipv6" as an indication for whether to use IPv4 or IPv4. The check
confused a definition like "[email protected]" to use IPv6 due to having
"ipv6" in the name.

To address the issue we changed the check to whether the prefix of
the provider name is "ipv6".

While doing so, we factored out this check into a own function taking
a ddns_info_* struct as an argument.
    @hrzlgnm
    hrzlgnm committed Sep 27, 2024
    Configuration menu
    Copy the full SHA
    46d2922 View commit details
    Browse the repository at this point in the history

  2. Fix logging of cache updates for providers having "v6" in their name 

    The original check, checked whether the alias name contained the
substring "v6". This lead to confusing logging messages for providers
having "v6" in their name. For example in definitions
like "[email protected]" or "[email protected]".
    @hrzlgnm
    hrzlgnm committed Sep 27, 2024
    Configuration menu
    Copy the full SHA
    4ea7896 View commit details
    Browse the repository at this point in the history