Fix nit

mullvad · Dec 24, 2023 · ab8c900 · ab8c900
1 parent 85572c1
commit ab8c900
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/talpid-core/src/tunnel_state_machine/connected_state.rs b/talpid-core/src/tunnel_state_machine/connected_state.rs
@@ -131,6 +131,11 @@ impl ConnectedState {
 
     fn get_firewall_policy(&self, shared_values: &SharedTunnelStateValues) -> FirewallPolicy {
         let endpoint = self.tunnel_parameters.get_next_hop_endpoint();
+
+        #[cfg(target_os = "windows")]
+        let clients = AllowedClients::from(TunnelMonitor::get_relay_client(&shared_values.resource_dir, params).into_iter().collect());
+
+        #[cfg(not(target_os = "windows"))]
         let clients = if self
             .tunnel_parameters
             .get_openvpn_local_proxy_settings()

diff --git a/talpid-core/src/tunnel_state_machine/connecting_state.rs b/talpid-core/src/tunnel_state_machine/connecting_state.rs
@@ -143,7 +143,7 @@ impl ConnectingState {
         let endpoint = params.get_next_hop_endpoint();
 
         #[cfg(target_os = "windows")]
-        let clients = AllowedClients::from(TunnelMonitor::get_relay_client(&shared_values.resource_dir, params));
+        let clients = AllowedClients::from(TunnelMonitor::get_relay_client(&shared_values.resource_dir, params).into_iter().collect());
 
         #[cfg(not(target_os = "windows"))]
         let clients = if params.get_openvpn_local_proxy_settings().is_some() {

diff --git a/talpid-types/src/net/mod.rs b/talpid-types/src/net/mod.rs
@@ -350,9 +350,9 @@ impl std::ops::Deref for AllowedClients {
 }
 
 #[cfg(windows)]
-impl From<Option<PathBuf>> for AllowedClients {
-    fn from(value: Option<PathBuf>) -> Self {
-        Self(value.iter().collect::<Vec<_>>().into())
+impl From<Vec<PathBuf>> for AllowedClients {
+    fn from(value: Vec<PathBuf>) -> Self {
+        Self(value.into())
     }
 }