Skip to content
Commit b7daee75 authored by Haochen Tong's avatar Haochen Tong Committed by Jan Grulich
Browse files

wireguard: Trim IP list before submitting to NM

Summary:
The `SimpleIpListValidator` validates each trimmed item in a comma-separated
list, but `WireGuardPeerWidget` sets `peerData` without trimming each item.
Untrimmed items are not accepted by NetworkManager, so the user might see NM
rejecting values that are valid in the peer settings dialog.

Fix this discrepancy by trimming each item before setting `peerData`.

Test Plan:
1. Type a comma-separated list of IP addresses, with spaces around comma, into
the "Allowed IPs" field of the WireGuard peer settings dialog.

2. Save the connection. No error should be reported by NM.

Reviewers: jgrulich

Reviewed By: jgrulich

Subscribers: usta, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D27661
parent 764cb342
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment