Bug 453453

Summary: IPv6 Address field not populated on existing WireGuard entries
Product: [Applications] systemsettings Reporter: Bruce Anderson <bcom>
Component: kcm_networkmanagementAssignee: Jan Grulich <jgrulich>
Status: RESOLVED FIXED    
Severity: normal CC: jgrulich, mss
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=457041
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Bruce Anderson 2022-05-06 05:06:32 UTC
SUMMARY
***
Opening an existing WireGuard connection which has an IPv6 method set to "Manual" and and address specified, no entries are populated in the Address table on the IPv6 tab of the editor. This makes it impossible to modify the connection because the "Apply" button never becomes active.
***


STEPS TO REPRODUCE
1. Create a WireGuard connection which includes a Manually supplied IPv6 address and save it.
2. Exit the configuration applet and start it again.
3. Go to the newly created WireGuard connection.

OBSERVED RESULT
Look at the IPv6 tab and verify that the previously entered address is not in the table. This also prevents saving any changes due without putting the IPv6 address back in because the entry is not valid.

EXPECTED RESULT
The IPv6 tab should have the previously entered address in the table. Should be able to save any changes without reentering the address.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: openSuse tumbleweed
KDE Plasma Version: 5.24.4
KDE Frameworks Version: 5.92.0
Qt Version: 5.15.2

ADDITIONAL INFORMATION
This works on a very old version (5.16.0) but this isn't something that I try to do very often so I don't know when it might have failed between then and now.
Comment 1 Bug Janitor Service 2023-06-08 05:01:54 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/networkmanager-qt/-/merge_requests/40
Comment 2 Bug Janitor Service 2024-09-10 04:54:40 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/networkmanager-qt/-/merge_requests/61
Comment 3 Christoph Cullmann 2024-09-21 17:18:43 UTC
Git commit 7e04bde94e629652987fab8e51750b3b8e0b1f14 by Christoph Cullmann, on behalf of J. T. Elscott.
Committed on 21/09/2024 at 17:16.
Pushed by cullmann into branch 'master'.

Correctly read manually-specified ipv6 addresses from Networkmanager

Copy+paste mistake was checking for the presence of DNS rather than addresses
Related: bug 476008

M  +1    -1    src/settings/ipv6setting.cpp

https://invent.kde.org/frameworks/networkmanager-qt/-/commit/7e04bde94e629652987fab8e51750b3b8e0b1f14