Skip to content

Commit

Permalink
Remove VPN server fetch at app launch (#2738)
Browse files Browse the repository at this point in the history
Task/Issue URL: https://app.asana.com/0/1199230911884351/1207229848611232/f
Tech Design URL:
CC:

Description:

This PR removes the server fetch at app launch.
  • Loading branch information
samsymons authored May 3, 2024
1 parent 0a211f2 commit 1342d8f
Showing 1 changed file with 0 additions and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@ final class NetworkProtectionAppEvents {
}

restartNetworkProtectionIfVersionChanged(using: loginItemsManager)
refreshNetworkProtectionServers()
}
}

Expand All @@ -97,19 +96,4 @@ final class NetworkProtectionAppEvents {
loginItemsManager.restartLoginItems(LoginItemsManager.networkProtectionLoginItems, log: .networkProtection)
}

/// Fetches a new list of VPN servers, and updates the existing set.
///
private func refreshNetworkProtectionServers() {
Task {
let serverCount: Int
do {
serverCount = try await NetworkProtectionDeviceManager.create().refreshServerList().count
} catch {
os_log("Failed to update DuckDuckGo VPN servers", log: .networkProtection, type: .error)
return
}

os_log("Successfully updated DuckDuckGo VPN servers; total server count = %{public}d", log: .networkProtection, serverCount)
}
}
}

0 comments on commit 1342d8f

Please sign in to comment.