summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorDavid Lönnhager <david.l@mullvad.net>2021-12-15 14:07:37 +0100
committerDavid Lönnhager <david.l@mullvad.net>2022-01-03 17:20:20 +0100
commit63c755c352ff6e5177a7787983322ce55bf80508 (patch)
tree51d12586cbbef7be2a47142b138daca109e29d76
parent4a56defff15ef7dae134f3db378b749def714a3b (diff)
downloadmullvadvpn-63c755c352ff6e5177a7787983322ce55bf80508.tar.xz
mullvadvpn-63c755c352ff6e5177a7787983322ce55bf80508.zip
Leave relay constraints alone when setting bridge state
-rw-r--r--mullvad-types/src/relay_constraints.rs24
-rw-r--r--mullvad-types/src/settings/mod.rs3
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