Bug 422889 - import of ovpn config file produces wrong result with multiple "remote" lines
Summary: import of ovpn config file produces wrong result with multiple "remote" lines
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_networkmanagement (show other bugs)
Version: unspecified
Platform: Fedora RPMs Linux
: NOR major
Target Milestone: ---
Assignee: Jan Grulich
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-06-12 18:56 UTC by Uri Simchoni
Modified: 2024-12-23 18:27 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
proposed git-am fix (1.09 KB, patch)
2020-06-12 18:59 UTC, Uri Simchoni 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Uri Simchoni 2020-06-12 18:56:57 UTC 
SUMMARY

When importing an ovpn file with multiple "remote" lines, if the first line denotes a UDP connection and a subsequent line denotes a TCP connection, the imported connection uses the IP address of the first line, but with TCP.

STEPS TO REPRODUCE

This happened to me with Company-produced ovpn file which obviously I cannot share, so following is a suggestion how to reproduce.
1. Obtain a valid .ovpn file with a UDP connection, which is known to work
2. Below the "remote" line, add another "remote" line with a different IP and "tcp" in the 4th column
3. Import the file using the connection editor

OBSERVED RESULT

The created connection contains the right (first) IP address, but is a TCP connection (when clicking "Advanced", "Use a TCP connection" is checked" and the connection doesn't work)

EXPECTED RESULT

The connection should be a UDP connection as with the unmodified file.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Fedora 32 KDE Plasma Spin
(available in About System)
KDE Plasma Version: 5.18.5
KDE Frameworks Version: 5.70.0
Qt Version: 5.13.2

ADDITIONAL INFORMATION
Comment 1 Uri Simchoni 2020-06-12 18:59:54 UTC 
Created attachment 129277 [details]
proposed git-am fix
Comment 2 Jan Grulich 2020-06-14 17:18:08 UTC 
Can you please submit your change in our Gitlab instance?

Link: https://invent.kde.org/plasma/plasma-nm/-/merge_requests
Comment 3 Uri Simchoni 2020-06-14 18:28:23 UTC 
(In reply to Jan Grulich from comment #2)
> Can you please submit your change in our Gitlab instance?
> 
> Link: https://invent.kde.org/plasma/plasma-nm/-/merge_requests

Done. Wasn't sure where this should go....
Comment 4 Nicolas Fella 2023-01-20 01:54:39 UTC 
https://invent.kde.org/plasma/plasma-nm/-/merge_requests/3 was the merge request