Skip to content

Commit

Permalink
Prevent resetting the VPN environment. (#738)
Browse files Browse the repository at this point in the history
Required:

Task/Issue URL: https://app.asana.com/0/1199230911884351/1206899355242110/f
iOS PR: duckduckgo/iOS#2623
macOS PR: duckduckgo/macos-browser#2477
What kind of version bump will this require?: Patch

Description:

This PR prevents the app from resetting the environment when uninstalling the VPN.

The environment is a debug-only setting, and uninstalling the browser is common to do when testing internally, so we should keep it persisted.
  • Loading branch information
samsymons authored Mar 22, 2024
1 parent 2f64ae6 commit 4009341
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -319,7 +319,7 @@ public actor NetworkProtectionDeviceManager: NetworkProtectionDeviceManagement {
dns: [DNSServer(address: server.serverInfo.internalIP)],
isKillSwitchEnabled: isKillSwitchEnabled)

return TunnelConfiguration(name: "Network Protection", interface: interface, peers: [peerConfiguration])
return TunnelConfiguration(name: "DuckDuckGo VPN", interface: interface, peers: [peerConfiguration])
}

func peerConfiguration(serverPublicKey: PublicKey, serverEndpoint: Endpoint) -> PeerConfiguration {
Expand Down
1 change: 0 additions & 1 deletion Sources/NetworkProtection/Settings/VPNSettings.swift
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,6 @@ public final class VPNSettings {
defaults.resetNetworkProtectionSettingNotifyStatusChanges()
defaults.resetNetworkProtectionSettingRegistrationKeyValidity()
defaults.resetNetworkProtectionSettingSelectedServer()
defaults.resetNetworkProtectionSettingSelectedEnvironment()
defaults.resetNetworkProtectionSettingShowInMenuBar()
}

Expand Down

0 comments on commit 4009341

Please sign in to comment.