ivpn / ios-app

Official IVPN iOS app
https://www.ivpn.net/apps-ios
GNU General Public License v3.0
439 stars 92 forks source link

Add “Block LAN traffic” action for untrusted networks #393

Closed jurajhilje closed 10 months ago

jurajhilje commented 10 months ago

Description

In the Network Protection settings, add “Block LAN traffic” action for untrusted networks. Note that when enabled, this option overrides the same option in Settings -> Advanced.

jurajhilje commented 10 months ago

@gorkapernas Available for QA in 2.11.1 (1)

gorkapernas commented 10 months ago

Verified on 2.11.1 (1), see my comments below.

Suggestion:

Image 2023-11-27 at 12 17 22 PM

Suggested text: "Network Protection" actions for untrusted networks will override this option.

jurajhilje commented 10 months ago

@gorkapernas New build 2.11.1 (3) is available, with fixes and improvements for your last comment.

gorkapernas commented 10 months ago

Verified on version 2.11.1 (3), the fixes and suggestions from my last comment have been implemented correctly. However, I'm still experiencing a couple of issues when enabling "Block LAN traffic" from the Advance settings.

  1. "Block LAN traffic" action for untrusted networks doesn't seem to override the same option from Advance settings. When both "Block LAN traffic" options are enabled, but without setting any trust status, the local area network is still blocked. In this case, since the network was not set as untrusted, LAN access should be allowed.

    Steps:

    • Enable Network Protection
    • Do not change any network trust status
    • Enable "Block LAN traffic" for untrusted networks in the Network Protection settings
    • Enable "Block LAN traffic" in Advance settings
    • Connect to the VPN
    • Observe that LAN access is blocked.
  2. Frequently, the app fails to connect with WireGuard when "Block LAN traffic" from Advance settings is enabled (same option in Network Protection is disabled)

    Steps:

    • Enable Network Protection
    • Enable "Block LAN traffic" in Advance settings
    • Connect to the VPN
jurajhilje commented 10 months ago

@gorkapernas New build 2.11.1 (6) is available:

jurajhilje commented 10 months ago

@gorkapernas New build 2.11.1 (7) is available:

gorkapernas commented 10 months ago

Verified on 2.11.1 (7), the tooltip has been updated correctly. There are no more open issues and the feature works as expected, only when "Block LAN traffic" action is enabled and the user connects to an untrusted network, the LAN traffic is blocked.