@@ -53,10 +53,9 @@ use talpid_types::{
53
53
ErrorExt ,
54
54
} ;
55
55
56
- /// [`WIREGUARD_RETRY_ORDER`] defines an ordered set of relay parameters which the relay selector
57
- /// should should prioritize on successive connection attempts. Note that these will *never*
58
- /// override user preferences. See [the documentation on `RelayQuery`][RelayQuery] for further
59
- /// details.
56
+ /// [`WIREGUARD_RETRY_ORDER`] defines an ordered set of relay parameters which the relay selector should
57
+ /// prioritize on successive connection attempts. Note that these will *never* override user
58
+ /// preferences. See [the documentation on `RelayQuery`][RelayQuery] for further details.
60
59
///
61
60
/// This list should be kept in sync with the expected behavior defined in `docs/relay-selector.md`
62
61
pub static WIREGUARD_RETRY_ORDER : LazyLock < Vec < RelayQuery > > = LazyLock :: new ( || {
@@ -82,8 +81,8 @@ pub static WIREGUARD_RETRY_ORDER: LazyLock<Vec<RelayQuery>> = LazyLock::new(|| {
82
81
]
83
82
} ) ;
84
83
85
- /// [`OPENVPN_RETRY_ORDER`] defines an ordered set of relay parameters which the relay selector
86
- /// should prioritize on successive connection attempts. Note that these will *never* override user
84
+ /// [`OPENVPN_RETRY_ORDER`] defines an ordered set of relay parameters which the relay selector should
85
+ /// prioritize on successive connection attempts. Note that these will *never* override user
87
86
/// preferences. See [the documentation on `RelayQuery`][RelayQuery] for further details.
88
87
///
89
88
/// This list should be kept in sync with the expected behavior defined in `docs/relay-selector.md`
@@ -694,10 +693,10 @@ impl RelaySelector {
694
693
parsed_relays : & RelayList ,
695
694
custom_lists : & CustomListsSettings ,
696
695
) -> Result < GetRelay , Error > {
697
- // FIXME: A bit of defensive programming - calling `get_wireguard_relay_inner` with a query
698
- // that doesn't specify Wireguard as the desired tunnel type is not valid and will
699
- // lead to unwanted behavior. This should be seen as a workaround, and it would be
700
- // nicer to lift this invariant to be checked by the type system instead.
696
+ // FIXME: A bit of defensive programming - calling `get_wireguard_relay_inner` with a query that
697
+ // doesn't specify Wireguard as the desired tunnel type is not valid and will lead
698
+ // to unwanted behavior. This should be seen as a workaround, and it would be nicer
699
+ // to lift this invariant to be checked by the type system instead.
701
700
let mut query = query. clone ( ) ;
702
701
query. set_tunnel_protocol ( TunnelType :: Wireguard ) ?;
703
702
Self :: get_wireguard_relay_inner ( & query, custom_lists, parsed_relays)
0 commit comments