Only reconnect if using relevant tunnel protocol

1 files changed, 31 insertions, 6 deletions

diff --git a/mullvad-daemon/src/lib.rs b/mullvad-daemon/src/lib.rs
index fe59f86d93..d116c635f0 100644
--- a/mullvad-daemon/src/lib.rs
+++ b/mullvad-daemon/src/lib.rs
@@ -67,7 +67,7 @@ use talpid_core::{
 #[cfg(target_os = "android")]
 use talpid_types::android::AndroidContext;
 use talpid_types::{
-    net::{openvpn, TransportProtocol, TunnelParameters},
+    net::{openvpn, TransportProtocol, TunnelParameters, TunnelType},
     tunnel::{ErrorStateCause, ParameterGenerationError, TunnelStateTransition},
     ErrorExt,
 };
@@ -1278,8 +1278,12 @@ where
                 Self::oneshot_send(tx, (), "set_openvpn_mssfix response");
                 if settings_changed {
                     self.event_listener.notify_settings(self.settings.clone());
-                    info!("Initiating tunnel restart because the OpenVPN mssfix setting changed");
-                    self.reconnect_tunnel();
+                    if let Some(TunnelType::OpenVpn) = self.get_connected_tunnel_type() {
+                        info!(
+                            "Initiating tunnel restart because the OpenVPN mssfix setting changed"
+                        );
+                        self.reconnect_tunnel();
+                    }
                 }
             }
             Err(e) => error!("{}", e.display_chain_with_msg("Unable to save settings")),
@@ -1358,8 +1362,12 @@ where
                 Self::oneshot_send(tx, (), "set_wireguard_mtu response");
                 if settings_changed {
                     self.event_listener.notify_settings(self.settings.clone());
-                    info!("Initiating tunnel restart because the WireGuard MTU setting changed");
-                    self.reconnect_tunnel();
+                    if let Some(TunnelType::Wireguard) = self.get_connected_tunnel_type() {
+                        info!(
+                            "Initiating tunnel restart because the WireGuard MTU setting changed"
+                        );
+                        self.reconnect_tunnel();
+                    }
                 }
             }
             Err(e) => error!("{}", e.display_chain_with_msg("Unable to save settings")),
@@ -1454,7 +1462,9 @@ where
                     self.account_history.insert(account_entry).map_err(|e| {
                         format!("Failed to add new wireguard key to account data: {}", e)
                     })?;
-                    self.reconnect_tunnel();
+                    if let Some(TunnelType::Wireguard) = self.get_connected_tunnel_type() {
+                        self.reconnect_tunnel();
+                    }
                     let keygen_event = KeygenEvent::NewKey(public_key);
                     self.event_listener.notify_key_event(keygen_event.clone());
 
@@ -1577,6 +1587,21 @@ where
         }
     }
 
+    fn get_connected_tunnel_type(&self) -> Option<TunnelType> {
+        use talpid_types::net::TunnelEndpoint;
+        use TunnelState::Connected;
+
+        if let Connected {
+            endpoint: TunnelEndpoint { tunnel_type, .. },
+            ..
+        } = self.tunnel_state
+        {
+            Some(tunnel_type)
+        } else {
+            None
+        }
+    }
+
     fn send_tunnel_command(&mut self, command: TunnelCommand) {
         self.tunnel_command_tx
             .unbounded_send(command)