diff options
| author | David Lönnhager <david.l@mullvad.net> | 2021-12-15 14:07:37 +0100 |
|---|---|---|
| committer | David Lönnhager <david.l@mullvad.net> | 2022-01-03 17:20:20 +0100 |
| commit | 63c755c352ff6e5177a7787983322ce55bf80508 (patch) | |
| tree | 51d12586cbbef7be2a47142b138daca109e29d76 | |
| parent | 4a56defff15ef7dae134f3db378b749def714a3b (diff) | |
| download | mullvadvpn-63c755c352ff6e5177a7787983322ce55bf80508.tar.xz mullvadvpn-63c755c352ff6e5177a7787983322ce55bf80508.zip | |
Leave relay constraints alone when setting bridge state
| -rw-r--r-- | mullvad-types/src/relay_constraints.rs | 24 | ||||
| -rw-r--r-- | mullvad-types/src/settings/mod.rs | 3 |
2 files changed, 0 insertions, 27 deletions
diff --git a/mullvad-types/src/relay_constraints.rs b/mullvad-types/src/relay_constraints.rs index 7789948797..18cdfda2d9 100644 --- a/mullvad-types/src/relay_constraints.rs +++ b/mullvad-types/src/relay_constraints.rs @@ -172,30 +172,6 @@ impl RelaySettings { }), } } - - pub(crate) fn ensure_bridge_compatibility(&mut self) { - match self { - RelaySettings::Normal(ref mut constraints) => { - if constraints.tunnel_protocol == Constraint::Only(TunnelType::Wireguard) { - constraints.tunnel_protocol = Constraint::Any; - } - if let Constraint::Only(TransportPort { - protocol: TransportProtocol::Udp, - .. - }) = constraints.openvpn_constraints.port - { - constraints.openvpn_constraints.port = Constraint::Any; - } - } - RelaySettings::CustomTunnelEndpoint(config) => { - if config.endpoint().protocol == TransportProtocol::Udp { - log::warn!( - "Using custom tunnel endpoint with UDP, bridges will likely not work" - ); - } - } - } - } } /// Limits the set of [`crate::relay_list::Relay`]s that a `RelaySelector` may select. diff --git a/mullvad-types/src/settings/mod.rs b/mullvad-types/src/settings/mod.rs index 0705764555..36981b82c7 100644 --- a/mullvad-types/src/settings/mod.rs +++ b/mullvad-types/src/settings/mod.rs @@ -194,9 +194,6 @@ impl Settings { pub fn set_bridge_state(&mut self, bridge_state: BridgeState) -> bool { if self.bridge_state != bridge_state { self.bridge_state = bridge_state; - if self.bridge_state == BridgeState::On { - self.relay_settings.ensure_bridge_compatibility(); - } true } else { false |
