Closed peno64 closed 1 year ago
This is not a good change. Most people (including myself) could not care what the server is, and displaying it fills up the notification box with scrolly text that detracts from the profile name and probably won't display unless the notification box timing is extended.
There's already a switch, display_location_on_connect, which changes the notification box to display more and leave it on the screen for longer for those users that want more information (which is a small number of them). I'd be ok to add this to that notification box (and because the code is a mess, it's in multiple places unfortunately).
Really though, I think this is just a reaction to the Nord situation? You can't force the server Nord uses (not true, you can use their API differently), and you can't guess which one will work with a particular service. If their random selection is not working, then I think the right answer is to download the set of server ovpn files that work and manage them via User Defined. This gives you the ability to filter automatically to the right server with each of the services/add-ons. Knowing which server is being used just means you need to mentally filter in your head and force a reconnect until it chooses the 'right' server.
With this change, when vpn manager has connected to a vpn server, the name of the server wil be shown in the popup