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

network-config-manager: Fix incorrect IPv6 config source #710

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

network-config-manager: Fix incorrect IPv6 config source #710

wants to merge 1 commit into from

Commits on Jul 23, 2024

  1. network-config-manager: Fix incorrect IPv6 config source 

    Issue - nmctl status <device> -j is showing ConfigSource as foreign
for all kind of address like static, dynamic and foreign.

Now on existing flow to get a link address info nmctl get the info
from dbus and parse the info similar for address as well. While parsing
the address from json object to ip, prefix length is not appended to address.
Hence ip matching with address always failing for IPv6, cause of this
ConfigSource always set to foreign regardless of address is static or dynamic.

Fix - Fixed in json_array_to_ip parsing to take care of prefix value as well
while returning the ip address after parsing the json object.
    @ntsbtz
    ntsbtz committed Jul 23, 2024
    Configuration menu
    Copy the full SHA
    821145b View commit details
    Browse the repository at this point in the history