diff options
| author | Emīls <emils@mullvad.net> | 2022-04-13 14:46:42 +0100 |
|---|---|---|
| committer | Emīls <emils@mullvad.net> | 2022-04-13 14:46:42 +0100 |
| commit | 53c95bb336012fe3a51f68f3f86170f882257646 (patch) | |
| tree | 1e6e09b9a6336a961b4de8dd62e8938b0d52a272 | |
| parent | cfa9c677cf86421373fccd46430e2428c4c070d8 (diff) | |
| download | mullvadvpn-53c95bb336012fe3a51f68f3f86170f882257646.tar.xz mullvadvpn-53c95bb336012fe3a51f68f3f86170f882257646.zip | |
Fix overzealous relay selector test
The relay selector test was testing if a WireGuard endpoint wasn't using
the same ports as TCP2UDP relays will be listening on. This doesn't make
much sense as a relay could very well be using those same ports, and in
some cases it was, thus making this test fail. The faulty assertion has
been removed.
| -rw-r--r-- | mullvad-daemon/src/relays/mod.rs | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/mullvad-daemon/src/relays/mod.rs b/mullvad-daemon/src/relays/mod.rs index b0a14d7ea3..1aa8001ba0 100644 --- a/mullvad-daemon/src/relays/mod.rs +++ b/mullvad-daemon/src/relays/mod.rs @@ -1661,7 +1661,6 @@ mod test { .get_tunnel_endpoint(&WIREGUARD_SINGLEHOP_CONSTRAINTS, BridgeState::Off, attempt) .expect("Failed to select a WireGuard relay"); assert!(result.entry_relay.is_none()); - assert!(!TCP2UDP_PORTS.contains(&result.endpoint.to_endpoint().address.port())); let (obfs_config, _obfs_relay) = relay_selector .get_obfuscator( |