TunnlTo / desktop-app

TunnlTo is a Windows WireGuard VPN client built for split tunnelling.
https://tunnl.to
1.11k stars 53 forks source link

replace line breakers when read rules for peer allowed/disallowed #180

Closed phatal closed 2 weeks ago

phatal commented 2 months ago

In the case of user input using a "line break" after the comma, there is a problem when separating folders/applications. This is easy to overlook from the user's point of view, and it causes the application to ignore the allowed/disallowed rules, effectively stopping performing its main function (I've seen a few issues that most likely happened for this reason) I've had the same problem myself, spending a whole day trying to figure out why tunnel splitting stopped working.

These 2 screenshots clearly demonstrate the problem. In the first case a space is used after the comma and everything is correct, but in the second case a "line break" is used, which leads to the addition of extra characters during reading and further error. It is very difficult for the user to notice his error. correct_split incorrect_split

brendanosborne commented 2 weeks ago

Thank you, i've added this to 1.0.8