Summary: | KDE Network Manager creates unusable name for wireguard and fails to save | ||
---|---|---|---|
Product: | [Plasma] plasma-nm | Reporter: | Matt Keith <rommy> |
Component: | applet | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | bernado_tornado, daniel-other+kdebug, nate, stalkerg, stephen.robinson |
Priority: | NOR | ||
Version: | 5.91.0 | ||
Target Milestone: | --- | ||
Platform: | Neon | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/plasma/plasma-nm/-/commit/3240d991fb635976dcbf8a40b9fa166210bc6881 | Version Fixed In: | 6.1 |
Description
Matt Keith
2023-12-30 03:42:40 UTC
I was able to reproduce this on kde plasma 6 beta 2 KDE Plasma Version: 5.91.90 KDE Freameworks Version: 5.248.0 QT Version: 6.6.1 Kernel Version: 6.2.0-39-generic (64-bit) Graphics Platform: Wayland Using the live demo installer. The issue is the same - the name is invalid - and the problem is the same - you lose everything trying to save. Also - the reproduction steps key and ip info are all false. But it will let you save the info. I of course found https://bugs.kde.org/show_bug.cgi?id=412795 which is for the same issue after creating this one. I feel mine is more useful so I will not mark it as duplicate myself A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-nm/-/merge_requests/322 *** Bug 412795 has been marked as a duplicate of this bug. *** *** Bug 414464 has been marked as a duplicate of this bug. *** I created a merge request to solve this by adding an interface name field to the wireguard configuration widget: https://invent.kde.org/plasma/plasma-nm/-/merge_requests/339 Git commit 3240d991fb635976dcbf8a40b9fa166210bc6881 by Nicolas Fella, on behalf of Stephen Robinson. Committed on 09/04/2024 at 21:46. Pushed by nicolasfella into branch 'master'. Add WireGuard config field for interface name Fixes ~~BUG: 412795~~ BUG: 479179 Previously the WireGuard configuration took the connection name as the interface name. This had a host of problems, especially that the default connection name was not valid for an interface name, and that an invalid interface name would result in losing all of your changes. This commit adds a required "Interface name" field (with proper validation). This behavior mirrors `nm-connection-editor`. ![New connection screenshot](/uploads/f4dbd88a73d207664ff1e096b30374e3/image.png) M +2 -0 libs/editor/CMakeLists.txt M +8 -3 libs/editor/connectioneditorbase.cpp A +38 -0 libs/editor/interfacenamevalidator.cpp [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +22 -0 libs/editor/interfacenamevalidator.h [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] M +24 -9 libs/editor/settings/ui/wireguardinterfacewidget.ui M +64 -4 libs/editor/settings/wireguardinterfacewidget.cpp M +6 -1 libs/editor/settings/wireguardinterfacewidget.h https://invent.kde.org/plasma/plasma-nm/-/commit/3240d991fb635976dcbf8a40b9fa166210bc6881 Is this bug https://bugs.kde.org/show_bug.cgi?id=484860 related? |