1
//! Experimental support for vanguards.
2
//!
3
//! For more information, see the [vanguards spec].
4
//!
5
//! [vanguards spec]: https://spec.torproject.org/vanguards-spec/index.html.
6

            
7
pub mod config;
8
mod err;
9
mod set;
10

            
11
use std::sync::{Arc, RwLock, Weak};
12
use std::time::{Duration, SystemTime};
13

            
14
use futures::stream::BoxStream;
15
use futures::{FutureExt as _, future};
16
use futures::{StreamExt as _, select_biased};
17
use postage::stream::Stream as _;
18
use postage::watch;
19
use tor_rtcompat::SpawnExt as _;
20

            
21
use tor_async_utils::PostageWatchSenderExt as _;
22
use tor_config::ReconfigureError;
23
use tor_error::{error_report, internal, into_internal};
24
use tor_netdir::{DirEvent, NetDir, NetDirProvider, Timeliness};
25
use tor_persist::{DynStorageHandle, StateMgr};
26
use tor_relay_selection::RelaySelector;
27
use tor_rtcompat::Runtime;
28
use tracing::{debug, info, instrument};
29

            
30
use crate::{RetireCircuits, VanguardMode};
31

            
32
use set::VanguardSets;
33

            
34
use crate::VanguardConfig;
35
pub use config::VanguardParams;
36
pub use err::VanguardMgrError;
37
pub use set::Vanguard;
38

            
39
/// The key used for storing the vanguard sets to persistent storage using `StateMgr`.
40
const STORAGE_KEY: &str = "vanguards";
41

            
42
/// The vanguard manager.
43
pub struct VanguardMgr<R: Runtime> {
44
    /// The mutable state.
45
    inner: RwLock<Inner>,
46
    /// The runtime.
47
    runtime: R,
48
    /// The persistent storage handle, used for writing the vanguard sets to disk
49
    /// if full vanguards are enabled.
50
    storage: DynStorageHandle<VanguardSets>,
51
}
52

            
53
/// The mutable inner state of [`VanguardMgr`].
54
struct Inner {
55
    /// The current vanguard parameters.
56
    params: VanguardParams,
57
    /// Whether to use full, lite, or no vanguards.
58
    ///
59
    // TODO(#1382): we should derive the mode from the
60
    // vanguards-enabled and vanguards-hs-service consensus params.
61
    mode: VanguardMode,
62
    /// The L2 and L3 vanguards.
63
    ///
64
    /// The L3 vanguards are only used if we are running in
65
    /// [`Full`](VanguardMode::Full) vanguard mode.
66
    /// Otherwise, the L3 set is not populated, or read from.
67
    ///
68
    /// If [`Full`](VanguardMode::Full) vanguard mode is enabled,
69
    /// the vanguard sets will be persisted to disk whenever
70
    /// vanuards are rotated, added, or removed.
71
    ///
72
    /// The vanguard sets are updated and persisted to storage by
73
    /// [`update_vanguard_sets`](Inner::update_vanguard_sets).
74
    ///
75
    /// If the `VanguardSets` change while we are in "lite" mode,
76
    /// the changes will not *not* be written to storage.
77
    /// If we later switch to "full" vanguards, those previous changes still
78
    /// won't be persisted to storage: we only flush to storage if the
79
    /// [`VanguardSets`] change *while* we are in "full" mode
80
    /// (changing the [`VanguardMode`] does not constitute a change in the `VanguardSets`).
81
    //
82
    // TODO HS-VANGUARDS: the correct behaviour here might be to never switch back to lite mode
83
    // after enabling full vanguards. If we do that, persisting the vanguard sets will be simpler,
84
    // as we can just unconditionally flush to storage if the vanguard mode is switched to full.
85
    // Right now we can't do that, because we don't remember the "mode":
86
    // we derive it on the fly from `has_onion_svc` and the current `VanguardParams`.
87
    //
88
    ///
89
    /// This is initialized with the vanguard sets read from the vanguard state file,
90
    /// if the file exists, or with a [`Default`] `VanguardSets`, if it doesn't.
91
    ///
92
    /// Note: The `VanguardSets` are read from the vanguard state file
93
    /// even if full vanguards are not enabled. They are *not*, however, written
94
    /// to the state file unless full vanguards are in use.
95
    vanguard_sets: VanguardSets,
96
    /// Whether we're running an onion service.
97
    ///
98
    // TODO(#1382): This should be used for deciding whether to use the `vanguards_hs_service` or the
99
    // `vanguards_enabled` [`NetParameter`](tor_netdir::params::NetParameters).
100
    #[allow(unused)]
101
    has_onion_svc: bool,
102
    /// A channel for sending VanguardConfig changes to the vanguard maintenance task.
103
    config_tx: watch::Sender<VanguardConfig>,
104
}
105

            
106
/// Whether the [`VanguardMgr::maintain_vanguard_sets`] task
107
/// should continue running or shut down.
108
///
109
/// Returned from [`VanguardMgr::run_once`].
110
#[derive(Copy, Clone, Debug)]
111
enum ShutdownStatus {
112
    /// Continue calling `run_once`.
113
    Continue,
114
    /// The `VanguardMgr` was dropped, terminate the task.
115
    Terminate,
116
}
117

            
118
impl<R: Runtime> VanguardMgr<R> {
119
    /// Create a new `VanguardMgr`.
120
    ///
121
    /// The `state_mgr` handle is used for persisting the "vanguards-full" guard pools to disk.
122
1822
    pub fn new<S>(
123
1822
        config: &VanguardConfig,
124
1822
        runtime: R,
125
1822
        state_mgr: S,
126
1822
        has_onion_svc: bool,
127
1822
    ) -> Result<Self, VanguardMgrError>
128
1822
    where
129
1822
        S: StateMgr + Send + Sync + 'static,
130
    {
131
        // Note: we start out with default vanguard params, but we adjust them
132
        // as soon as we obtain a NetDir (see Self::run_once()).
133
1822
        let params = VanguardParams::default();
134
1822
        let storage: DynStorageHandle<VanguardSets> = state_mgr.create_handle(STORAGE_KEY);
135

            
136
1822
        let vanguard_sets = match storage.load()? {
137
8
            Some(mut sets) => {
138
8
                info!("Loading vanguards from vanguard state file");
139
                // Discard the now-expired the vanguards
140
8
                let now = runtime.wallclock();
141
8
                let _ = sets.remove_expired(now);
142
8
                sets
143
            }
144
            None => {
145
1810
                debug!("Vanguard state file not found, selecting new vanguards");
146
                // Initially, all sets have a target size of 0.
147
                // This is OK because the target is only used for repopulating the vanguard sets,
148
                // and we can't repopulate the sets without a netdir.
149
                // The target gets adjusted once we obtain a netdir.
150
1810
                Default::default()
151
            }
152
        };
153

            
154
1818
        let (config_tx, _config_rx) = watch::channel();
155
1818
        let inner = Inner {
156
1818
            params,
157
1818
            mode: config.mode(),
158
1818
            vanguard_sets,
159
1818
            has_onion_svc,
160
1818
            config_tx,
161
1818
        };
162

            
163
1818
        Ok(Self {
164
1818
            inner: RwLock::new(inner),
165
1818
            runtime,
166
1818
            storage,
167
1818
        })
168
1822
    }
169

            
170
    /// Launch the vanguard pool management tasks.
171
    ///
172
    /// These run until the `VanguardMgr` is dropped.
173
    //
174
    // This spawns [`VanguardMgr::maintain_vanguard_sets`].
175
    #[instrument(level = "trace", skip_all)]
176
84
    pub fn launch_background_tasks(
177
84
        self: &Arc<Self>,
178
84
        netdir_provider: &Arc<dyn NetDirProvider>,
179
84
    ) -> Result<(), VanguardMgrError>
180
84
    where
181
84
        R: Runtime,
182
    {
183
84
        let netdir_provider = Arc::clone(netdir_provider);
184
84
        let config_rx = self
185
84
            .inner
186
84
            .write()
187
84
            .expect("poisoned lock")
188
84
            .config_tx
189
84
            .subscribe();
190
84
        self.runtime
191
84
            .spawn(Self::maintain_vanguard_sets(
192
84
                Arc::downgrade(self),
193
84
                Arc::downgrade(&netdir_provider),
194
84
                config_rx,
195
            ))
196
84
            .map_err(|e| VanguardMgrError::Spawn(Arc::new(e)))?;
197

            
198
84
        Ok(())
199
84
    }
200

            
201
    /// Replace the configuration in this `VanguardMgr` with the specified `config`.
202
16
    pub fn reconfigure(&self, config: &VanguardConfig) -> Result<RetireCircuits, ReconfigureError> {
203
        // TODO(#1382): abolish VanguardConfig and derive the mode from the VanguardParams
204
        // and has_onion_svc instead.
205
        //
206
        // TODO(#1382): update has_onion_svc if the new config enables onion svc usage
207
        //
208
        // Perhaps we should always escalate to Full if we start running an onion service,
209
        // but not decessarily downgrade to lite if we stop.
210
        // See <https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2083#note_3018173>
211
16
        let mut inner = self.inner.write().expect("poisoned lock");
212
16
        let new_mode = config.mode();
213
16
        if new_mode != inner.mode {
214
16
            inner.mode = new_mode;
215

            
216
            // Wake up the maintenance task to replenish the vanguard pools.
217
16
            inner.config_tx.maybe_send(|_| config.clone());
218

            
219
16
            Ok(RetireCircuits::All)
220
        } else {
221
            Ok(RetireCircuits::None)
222
        }
223
16
    }
224

            
225
    /// Return a [`Vanguard`] relay for use in the specified layer.
226
    ///
227
    /// The `relay_selector` must exclude the relays that would neighbor this vanguard
228
    /// in the path.
229
    ///
230
    /// Specifically, it should exclude
231
    ///   * the last relay in the path (the one immediately preceding the vanguard): the same relay
232
    ///     cannot be used in consecutive positions in the path (a relay won't let you extend the
233
    ///     circuit to itself).
234
    ///   * the penultimate relay of the path, if there is one: relays don't allow extending the
235
    ///     circuit to their previous hop
236
    ///
237
    /// If [`Full`](VanguardMode::Full) vanguards are in use, this function can be used
238
    /// for selecting both [`Layer2`](Layer::Layer2) and [`Layer3`](Layer::Layer3) vanguards.
239
    ///
240
    /// If [`Lite`](VanguardMode::Lite) vanguards are in use, this function can only be used
241
    /// for selecting [`Layer2`](Layer::Layer2) vanguards.
242
    /// It will return an error if a [`Layer3`](Layer::Layer3) is requested.
243
    ///
244
    /// Returns an error if vanguards are disabled.
245
    ///
246
    /// Returns a [`NoSuitableRelay`](VanguardMgrError::NoSuitableRelay) error
247
    /// if none of our vanguards satisfy the `layer` and `neighbor_exclusion` requirements.
248
    ///
249
    /// Returns a [`BootstrapRequired`](VanguardMgrError::BootstrapRequired) error
250
    /// if called before the vanguard manager has finished bootstrapping,
251
    /// or if all the vanguards have become unusable
252
    /// (by expiring or no longer being listed in the consensus)
253
    /// and we are unable to replenish them.
254
    ///
255
    ///  ### Example
256
    ///
257
    ///  If the partially built path is of the form `G - L2` and we are selecting the L3 vanguard,
258
    ///  the `RelayExclusion` should contain `G` and `L2` (to prevent building a path of the form
259
    ///  `G - L2 - G`, or `G - L2 - L2`).
260
    ///
261
    ///  If the path only contains the L1 guard (`G`), then the `RelayExclusion` should only
262
    ///  exclude `G`.
263
116
    pub fn select_vanguard<'a, Rng: rand::Rng>(
264
116
        &self,
265
116
        rng: &mut Rng,
266
116
        netdir: &'a NetDir,
267
116
        layer: Layer,
268
116
        relay_selector: &RelaySelector<'a>,
269
116
    ) -> Result<Vanguard<'a>, VanguardMgrError> {
270
        use VanguardMode::*;
271

            
272
116
        let inner = self.inner.read().expect("poisoned lock");
273

            
274
        // All our vanguard sets are empty. This means select_vanguards() was called before
275
        // maintain_vanguard_sets() managed to obtain a netdir and populate the vanguard sets,
276
        // or all the vanguards have become unusable and we have been unable to replenish them.
277
116
        if inner.vanguard_sets.l2().is_empty() && inner.vanguard_sets.l3().is_empty() {
278
4
            return Err(VanguardMgrError::BootstrapRequired {
279
4
                action: "select vanguard",
280
4
            });
281
112
        }
282

            
283
104
        let relay =
284
112
            match (layer, inner.mode) {
285
64
                (Layer::Layer2, Full) | (Layer::Layer2, Lite) => inner
286
64
                    .vanguard_sets
287
64
                    .l2()
288
64
                    .pick_relay(rng, netdir, relay_selector),
289
                (Layer::Layer3, Full) => {
290
40
                    inner
291
40
                        .vanguard_sets
292
40
                        .l3()
293
40
                        .pick_relay(rng, netdir, relay_selector)
294
                }
295
                _ => {
296
8
                    return Err(VanguardMgrError::LayerNotSupported {
297
8
                        layer,
298
8
                        mode: inner.mode,
299
8
                    });
300
                }
301
            };
302

            
303
104
        relay.ok_or(VanguardMgrError::NoSuitableRelay(layer))
304
116
    }
305

            
306
    /// The vanguard set management task.
307
    ///
308
    /// This is a background task that:
309
    /// * removes vanguards from the L2 and L3 vanguard sets when they expire
310
    /// * ensures the vanguard sets are repopulated with new vanguards
311
    ///   when the number of vanguards drops below a certain threshold
312
    /// * handles `NetDir` changes, updating the vanguard set sizes as needed
313
84
    async fn maintain_vanguard_sets(
314
84
        mgr: Weak<Self>,
315
84
        netdir_provider: Weak<dyn NetDirProvider>,
316
84
        mut config_rx: watch::Receiver<VanguardConfig>,
317
84
    ) {
318
84
        let mut netdir_events = match netdir_provider.upgrade() {
319
80
            Some(provider) => provider.events(),
320
            None => {
321
4
                return;
322
            }
323
        };
324

            
325
        loop {
326
284
            match Self::run_once(
327
284
                Weak::clone(&mgr),
328
284
                Weak::clone(&netdir_provider),
329
284
                &mut netdir_events,
330
284
                &mut config_rx,
331
            )
332
284
            .await
333
            {
334
204
                Ok(ShutdownStatus::Continue) => continue,
335
                Ok(ShutdownStatus::Terminate) => {
336
                    debug!("Vanguard manager is shutting down");
337
                    break;
338
                }
339
4
                Err(e) => {
340
4
                    error_report!(e, "Vanguard manager crashed");
341
4
                    break;
342
                }
343
            }
344
        }
345
8
    }
346

            
347
    /// Wait until a vanguard expires or until there is a new [`NetDir`].
348
    ///
349
    /// This populates the L2 and L3 vanguard sets,
350
    /// and rotates the vanguards when their lifetime expires.
351
    ///
352
    /// Note: the L3 set is only populated with vanguards if
353
    /// [`Full`](VanguardMode::Full) vanguards are enabled.
354
284
    async fn run_once(
355
284
        mgr: Weak<Self>,
356
284
        netdir_provider: Weak<dyn NetDirProvider>,
357
284
        netdir_events: &mut BoxStream<'static, DirEvent>,
358
284
        config_rx: &mut watch::Receiver<VanguardConfig>,
359
284
    ) -> Result<ShutdownStatus, VanguardMgrError> {
360
284
        let (mgr, netdir_provider) = match (mgr.upgrade(), netdir_provider.upgrade()) {
361
284
            (Some(mgr), Some(netdir_provider)) => (mgr, netdir_provider),
362
            _ => return Ok(ShutdownStatus::Terminate),
363
        };
364

            
365
284
        let now = mgr.runtime.wallclock();
366
284
        let next_to_expire = mgr.rotate_expired(&netdir_provider, now)?;
367
        // A future that sleeps until the next vanguard expires
368
284
        let sleep_fut = async {
369
200
            if let Some(dur) = next_to_expire {
370
124
                let () = mgr.runtime.sleep(dur).await;
371
            } else {
372
76
                future::pending::<()>().await;
373
            }
374
16
        };
375

            
376
284
        select_biased! {
377
284
            event = netdir_events.next().fuse() => {
378
104
                if let Some(DirEvent::NewConsensus) = event {
379
104
                    let netdir = netdir_provider.netdir(Timeliness::Timely)?;
380
104
                    mgr.inner.write().expect("poisoned lock")
381
104
                        .update_vanguard_sets(&mgr.runtime, &mgr.storage, &netdir)?;
382
                }
383

            
384
100
                Ok(ShutdownStatus::Continue)
385
            },
386
284
            _config = config_rx.recv().fuse() => {
387
88
                if let Some(netdir) = Self::timely_netdir(&netdir_provider)? {
388
                    // If we have a NetDir, replenish the vanguard sets that don't have enough vanguards.
389
                    //
390
                    // For example, if the config change enables full vanguards for the first time,
391
                    // this will cause the L3 vanguard set to be populated.
392
8
                    mgr.inner.write().expect("poisoned lock")
393
8
                        .update_vanguard_sets(&mgr.runtime, &mgr.storage, &netdir)?;
394
80
                }
395

            
396
88
                Ok(ShutdownStatus::Continue)
397
            },
398
284
            () = sleep_fut.fuse() => {
399
                // A vanguard expired, time to run the cleanup
400
16
                Ok(ShutdownStatus::Continue)
401
            },
402
        }
403
208
    }
404

            
405
    /// Return a timely `NetDir`, if one is available.
406
    ///
407
    /// Returns `None` if no directory information is available.
408
372
    fn timely_netdir(
409
372
        netdir_provider: &Arc<dyn NetDirProvider>,
410
372
    ) -> Result<Option<Arc<NetDir>>, VanguardMgrError> {
411
        use tor_netdir::Error as NetDirError;
412

            
413
372
        match netdir_provider.netdir(Timeliness::Timely) {
414
132
            Ok(netdir) => Ok(Some(netdir)),
415
240
            Err(NetDirError::NoInfo) | Err(NetDirError::NotEnoughInfo) => Ok(None),
416
            Err(e) => Err(e.into()),
417
        }
418
372
    }
419

            
420
    /// Rotate the vanguards that have expired,
421
    /// returning how long until the next vanguard will expire,
422
    /// or `None` if there are no vanguards in any of our sets.
423
284
    fn rotate_expired(
424
284
        &self,
425
284
        netdir_provider: &Arc<dyn NetDirProvider>,
426
284
        now: SystemTime,
427
284
    ) -> Result<Option<Duration>, VanguardMgrError> {
428
284
        let mut inner = self.inner.write().expect("poisoned lock");
429
284
        let inner = &mut *inner;
430

            
431
284
        let vanguard_sets = &mut inner.vanguard_sets;
432
284
        let expired_count = vanguard_sets.remove_expired(now);
433

            
434
284
        if expired_count > 0 {
435
16
            info!("Rotating vanguards");
436
268
        }
437

            
438
284
        if let Some(netdir) = Self::timely_netdir(netdir_provider)? {
439
            // If we have a NetDir, replenish the vanguard sets that don't have enough vanguards.
440
124
            inner.update_vanguard_sets(&self.runtime, &self.storage, &netdir)?;
441
160
        }
442

            
443
284
        let Some(expiry) = inner.vanguard_sets.next_expiry() else {
444
            // Both vanguard sets are empty
445
152
            return Ok(None);
446
        };
447

            
448
132
        expiry
449
132
            .duration_since(now)
450
132
            .map_err(|_| internal!("when > now, but now is later than when?!").into())
451
132
            .map(Some)
452
284
    }
453

            
454
    /// Get the current [`VanguardMode`].
455
144
    pub fn mode(&self) -> VanguardMode {
456
144
        self.inner.read().expect("poisoned lock").mode
457
144
    }
458
}
459

            
460
impl Inner {
461
    /// Update the vanguard sets, handling any potential vanguard parameter changes.
462
    ///
463
    /// This updates the [`VanguardSets`]s based on the [`VanguardParams`]
464
    /// derived from the new `NetDir`, replenishing the sets if necessary.
465
    ///
466
    /// NOTE: if the new `VanguardParams` specify different lifetime ranges
467
    /// than the previous `VanguardParams`, the new lifetime requirements only
468
    /// apply to newly selected vanguards. They are **not** retroactively applied
469
    /// to our existing vanguards.
470
    //
471
    // TODO(#1352): we might want to revisit this decision.
472
    // We could, for example, adjust the lifetime of our existing vanguards
473
    // to comply with the new lifetime requirements.
474
236
    fn update_vanguard_sets<R: Runtime>(
475
236
        &mut self,
476
236
        runtime: &R,
477
236
        storage: &DynStorageHandle<VanguardSets>,
478
236
        netdir: &Arc<NetDir>,
479
236
    ) -> Result<(), VanguardMgrError> {
480
236
        let params = VanguardParams::try_from(netdir.params())
481
236
            .map_err(into_internal!("invalid NetParameters"))?;
482

            
483
        // Update our params with the new values.
484
236
        self.update_params(params.clone());
485

            
486
236
        self.vanguard_sets.remove_unlisted(netdir);
487

            
488
        // If we loaded some vanguards from persistent storage but we still need more,
489
        // we select them here.
490
        //
491
        // If full vanguards are not enabled and we started with an empty (default)
492
        // vanguard set, we populate the sets here.
493
        //
494
        // If we have already populated the vanguard sets in a previous iteration,
495
        // this will ensure they have enough vanguards.
496
236
        self.vanguard_sets
497
236
            .replenish_vanguards(runtime, netdir, &params, self.mode)?;
498

            
499
        // Flush the vanguard sets to disk.
500
236
        self.flush_to_storage(storage)?;
501

            
502
232
        Ok(())
503
236
    }
504

            
505
    /// Update our vanguard params.
506
2644
    fn update_params(&mut self, new_params: VanguardParams) {
507
2644
        self.params = new_params;
508
2644
    }
509

            
510
    /// Flush the vanguard sets to storage, if the mode is "vanguards-full".
511
2644
    fn flush_to_storage(
512
2644
        &self,
513
2644
        storage: &DynStorageHandle<VanguardSets>,
514
2644
    ) -> Result<(), VanguardMgrError> {
515
2644
        match self.mode {
516
1184
            VanguardMode::Lite | VanguardMode::Disabled => Ok(()),
517
            VanguardMode::Full => {
518
1460
                debug!("The vanguards may have changed; flushing to vanguard state file");
519
1460
                Ok(storage.store(&self.vanguard_sets)?)
520
            }
521
        }
522
2644
    }
523
}
524

            
525
#[cfg(any(test, feature = "testing"))]
526
use {
527
    tor_config::ExplicitOrAuto, tor_netdir::testprovider::TestNetDirProvider,
528
    tor_persist::TestingStateMgr, tor_rtmock::MockRuntime,
529
};
530

            
531
/// Helpers for tests involving vanguards
532
#[cfg(any(test, feature = "testing"))]
533
impl VanguardMgr<MockRuntime> {
534
    /// Create a new VanguardMgr for testing.
535
1284
    pub fn new_testing(
536
1284
        rt: &MockRuntime,
537
1284
        mode: VanguardMode,
538
1284
    ) -> Result<Arc<VanguardMgr<MockRuntime>>, VanguardMgrError> {
539
1284
        let config = VanguardConfig {
540
1284
            mode: ExplicitOrAuto::Explicit(mode),
541
1284
        };
542
1284
        let statemgr = TestingStateMgr::new();
543
1284
        let lock = statemgr.try_lock()?;
544
1284
        assert!(lock.held());
545
        // TODO(#1382): has_onion_svc doesn't matter right now
546
1284
        let has_onion_svc = false;
547
1284
        Ok(Arc::new(VanguardMgr::new(
548
1284
            &config,
549
1284
            rt.clone(),
550
1284
            statemgr,
551
1284
            has_onion_svc,
552
        )?))
553
1284
    }
554

            
555
    /// Wait until the vanguardmgr has populated its vanguard sets.
556
    ///
557
    /// Returns a [`TestNetDirProvider`] that can be used to notify
558
    /// the `VanguardMgr` of netdir changes.
559
1284
    pub async fn init_vanguard_sets(
560
1284
        self: &Arc<VanguardMgr<MockRuntime>>,
561
1284
        netdir: &NetDir,
562
1324
    ) -> Result<Arc<TestNetDirProvider>, VanguardMgrError> {
563
80
        let netdir_provider = Arc::new(TestNetDirProvider::new());
564
80
        self.launch_background_tasks(&(netdir_provider.clone() as Arc<dyn NetDirProvider>))?;
565
80
        self.runtime.progress_until_stalled().await;
566

            
567
        // Call set_netdir_and_notify to trigger an event
568
80
        netdir_provider
569
80
            .set_netdir_and_notify(Arc::new(netdir.clone()))
570
80
            .await;
571

            
572
        // Wait until the vanguard mgr has finished handling the netdir event.
573
80
        self.runtime.progress_until_stalled().await;
574

            
575
80
        Ok(netdir_provider)
576
80
    }
577
}
578

            
579
/// The vanguard layer.
580
#[derive(Debug, Clone, Copy, PartialEq)] //
581
#[derive(derive_more::Display)] //
582
#[non_exhaustive]
583
pub enum Layer {
584
    /// L2 vanguard.
585
    #[display("layer 2")]
586
    Layer2,
587
    /// L3 vanguard.
588
    #[display("layer 3")]
589
    Layer3,
590
}
591

            
592
#[cfg(test)]
593
mod test {
594
    // @@ begin test lint list maintained by maint/add_warning @@
595
    #![allow(clippy::bool_assert_comparison)]
596
    #![allow(clippy::clone_on_copy)]
597
    #![allow(clippy::dbg_macro)]
598
    #![allow(clippy::mixed_attributes_style)]
599
    #![allow(clippy::print_stderr)]
600
    #![allow(clippy::print_stdout)]
601
    #![allow(clippy::single_char_pattern)]
602
    #![allow(clippy::unwrap_used)]
603
    #![allow(clippy::unchecked_time_subtraction)]
604
    #![allow(clippy::useless_vec)]
605
    #![allow(clippy::needless_pass_by_value)]
606
    #![allow(clippy::string_slice)] // See arti#2571
607
    //! <!-- @@ end test lint list maintained by maint/add_warning @@ -->
608

            
609
    use std::{fmt, time};
610

            
611
    use set::TimeBoundVanguard;
612
    use tor_config::ExplicitOrAuto;
613
    use tor_relay_selection::RelayExclusion;
614

            
615
    use super::*;
616

            
617
    use Layer::*;
618
    use tor_basic_utils::test_rng::testing_rng;
619
    use tor_linkspec::{HasRelayIds, RelayIds};
620
    use tor_netdir::{
621
        testnet::{self, construct_custom_netdir_with_params},
622
        testprovider::TestNetDirProvider,
623
    };
624
    use tor_persist::FsStateMgr;
625
    use tor_rtmock::MockRuntime;
626

            
627
    use itertools::Itertools;
628

            
629
    /// Enable lite vanguards for onion services.
630
    const ENABLE_LITE_VANGUARDS: [(&str, i32); 1] = [("vanguards-hs-service", 1)];
631

            
632
    /// Enable full vanguards for hidden services.
633
    const ENABLE_FULL_VANGUARDS: [(&str, i32); 1] = [("vanguards-hs-service", 2)];
634

            
635
    /// A valid vanguard state file.
636
    const VANGUARDS_JSON: &str = include_str!("../testdata/vanguards.json");
637

            
638
    /// A invalid vanguard state file.
639
    const INVALID_VANGUARDS_JSON: &str = include_str!("../testdata/vanguards_invalid.json");
640

            
641
    /// Create the `StateMgr`, populating the vanguards.json state file with the specified JSON string.
642
    fn state_dir_with_vanguards(vanguards_json: &str) -> (FsStateMgr, tempfile::TempDir) {
643
        let dir = tempfile::TempDir::new().unwrap();
644
        std::fs::create_dir_all(dir.path().join("state")).unwrap();
645
        std::fs::write(dir.path().join("state/vanguards.json"), vanguards_json).unwrap();
646

            
647
        let statemgr = FsStateMgr::from_path_and_mistrust(
648
            dir.path(),
649
            &fs_mistrust::Mistrust::new_dangerously_trust_everyone(),
650
        )
651
        .unwrap();
652

            
653
        (statemgr, dir)
654
    }
655

            
656
    impl fmt::Debug for Vanguard<'_> {
657
        fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
658
            f.debug_struct("Vanguard").finish()
659
        }
660
    }
661

            
662
    impl Inner {
663
        /// Return the L2 vanguard set.
664
        pub(super) fn l2_vanguards(&self) -> &Vec<TimeBoundVanguard> {
665
            self.vanguard_sets.l2_vanguards()
666
        }
667

            
668
        /// Return the L3 vanguard set.
669
        pub(super) fn l3_vanguards(&self) -> &Vec<TimeBoundVanguard> {
670
            self.vanguard_sets.l3_vanguards()
671
        }
672
    }
673

            
674
    /// Return a maximally permissive RelaySelector for a vanguard.
675
    fn permissive_selector() -> RelaySelector<'static> {
676
        RelaySelector::new(
677
            tor_relay_selection::RelayUsage::vanguard(),
678
            RelayExclusion::no_relays_excluded(),
679
        )
680
    }
681

            
682
    /// Look up the vanguard in the specified VanguardSet.
683
    fn find_in_set<R: Runtime>(
684
        relay_ids: &RelayIds,
685
        mgr: &VanguardMgr<R>,
686
        layer: Layer,
687
    ) -> Option<TimeBoundVanguard> {
688
        let inner = mgr.inner.read().unwrap();
689

            
690
        let vanguards = match layer {
691
            Layer2 => inner.l2_vanguards(),
692
            Layer3 => inner.l3_vanguards(),
693
        };
694

            
695
        // Look up the TimeBoundVanguard that corresponds to this Vanguard,
696
        // and figure out its expiry.
697
        vanguards.iter().find(|v| v.id == *relay_ids).cloned()
698
    }
699

            
700
    /// Get the total number of vanguard entries (L2 + L3).
701
    fn vanguard_count<R: Runtime>(mgr: &VanguardMgr<R>) -> usize {
702
        let inner = mgr.inner.read().unwrap();
703
        inner.l2_vanguards().len() + inner.l3_vanguards().len()
704
    }
705

            
706
    /// Return a `Duration` representing how long until this vanguard expires.
707
    fn duration_until_expiry<R: Runtime>(
708
        relay_ids: &RelayIds,
709
        mgr: &VanguardMgr<R>,
710
        runtime: &R,
711
        layer: Layer,
712
    ) -> Duration {
713
        // Look up the TimeBoundVanguard that corresponds to this Vanguard,
714
        // and figure out its expiry.
715
        let vanguard = find_in_set(relay_ids, mgr, layer).unwrap();
716

            
717
        vanguard
718
            .when
719
            .duration_since(runtime.wallclock())
720
            .unwrap_or_default()
721
    }
722

            
723
    /// Assert the lifetime of the specified `vanguard` is within the bounds of its `layer`.
724
    fn assert_expiry_in_bounds<R: Runtime>(
725
        vanguard: &Vanguard<'_>,
726
        mgr: &VanguardMgr<R>,
727
        runtime: &R,
728
        params: &VanguardParams,
729
        layer: Layer,
730
    ) {
731
        let (min, max) = match layer {
732
            Layer2 => (params.l2_lifetime_min(), params.l2_lifetime_max()),
733
            Layer3 => (params.l3_lifetime_min(), params.l3_lifetime_max()),
734
        };
735

            
736
        let vanguard = RelayIds::from_relay_ids(vanguard.relay());
737
        // This is not exactly the lifetime of the vanguard,
738
        // but rather the time left until it expires (but it's close enough for our purposes).
739
        let lifetime = duration_until_expiry(&vanguard, mgr, runtime, layer);
740

            
741
        assert!(
742
            lifetime >= min && lifetime <= max,
743
            "lifetime {lifetime:?} not between {min:?} and {max:?}",
744
        );
745
    }
746

            
747
    /// Assert that the vanguard manager's pools are empty.
748
    fn assert_sets_empty<R: Runtime>(vanguardmgr: &VanguardMgr<R>) {
749
        let inner = vanguardmgr.inner.read().unwrap();
750
        // The sets are initially empty, and the targets are set to 0
751
        assert_eq!(inner.vanguard_sets.l2_vanguards_deficit(), 0);
752
        assert_eq!(inner.vanguard_sets.l3_vanguards_deficit(), 0);
753
        assert_eq!(vanguard_count(vanguardmgr), 0);
754
    }
755

            
756
    /// Assert that the vanguard manager's pools have been filled.
757
    fn assert_sets_filled<R: Runtime>(vanguardmgr: &VanguardMgr<R>, params: &VanguardParams) {
758
        let inner = vanguardmgr.inner.read().unwrap();
759
        let l2_pool_size = params.l2_pool_size();
760
        // The sets are initially empty
761
        assert_eq!(inner.vanguard_sets.l2_vanguards_deficit(), 0);
762

            
763
        if inner.mode == VanguardMode::Full {
764
            assert_eq!(inner.vanguard_sets.l3_vanguards_deficit(), 0);
765
            let l3_pool_size = params.l3_pool_size();
766
            assert_eq!(vanguard_count(vanguardmgr), l2_pool_size + l3_pool_size);
767
        }
768
    }
769

            
770
    /// Assert the target size of the specified vanguard set matches the target from `params`.
771
    fn assert_set_vanguards_targets_match_params<R: Runtime>(
772
        mgr: &VanguardMgr<R>,
773
        params: &VanguardParams,
774
    ) {
775
        let inner = mgr.inner.read().unwrap();
776
        assert_eq!(
777
            inner.vanguard_sets.l2_vanguards_target(),
778
            params.l2_pool_size()
779
        );
780
        if inner.mode == VanguardMode::Full {
781
            assert_eq!(
782
                inner.vanguard_sets.l3_vanguards_target(),
783
                params.l3_pool_size()
784
            );
785
        }
786
    }
787

            
788
    #[test]
789
    fn full_vanguards_disabled() {
790
        MockRuntime::test_with_various(|rt| async move {
791
            let vanguardmgr = VanguardMgr::new_testing(&rt, VanguardMode::Lite).unwrap();
792
            let netdir = testnet::construct_netdir().unwrap_if_sufficient().unwrap();
793
            let mut rng = testing_rng();
794
            // Wait until the vanguard manager has bootstrapped
795
            // (otherwise we'll get a BootstrapRequired error)
796
            let _netdir_provider = vanguardmgr.init_vanguard_sets(&netdir).await.unwrap();
797

            
798
            // Cannot select an L3 vanguard when running in "Lite" mode.
799
            let err = vanguardmgr
800
                .select_vanguard(&mut rng, &netdir, Layer3, &permissive_selector())
801
                .unwrap_err();
802
            assert!(
803
                matches!(
804
                    err,
805
                    VanguardMgrError::LayerNotSupported {
806
                        layer: Layer::Layer3,
807
                        mode: VanguardMode::Lite
808
                    }
809
                ),
810
                "{err}"
811
            );
812
        });
813
    }
814

            
815
    #[test]
816
    fn background_task_not_spawned() {
817
        MockRuntime::test_with_various(|rt| async move {
818
            let vanguardmgr = VanguardMgr::new_testing(&rt, VanguardMode::Lite).unwrap();
819
            let netdir = testnet::construct_netdir().unwrap_if_sufficient().unwrap();
820
            let mut rng = testing_rng();
821

            
822
            // The sets are initially empty
823
            assert_sets_empty(&vanguardmgr);
824

            
825
            // VanguardMgr::launch_background tasks was not called, so select_vanguard will return
826
            // an error (because the vanguard sets are empty)
827
            let err = vanguardmgr
828
                .select_vanguard(&mut rng, &netdir, Layer2, &permissive_selector())
829
                .unwrap_err();
830

            
831
            assert!(
832
                matches!(
833
                    err,
834
                    VanguardMgrError::BootstrapRequired {
835
                        action: "select vanguard"
836
                    }
837
                ),
838
                "{err:?}"
839
            );
840
        });
841
    }
842

            
843
    #[test]
844
    fn select_vanguards() {
845
        MockRuntime::test_with_various(|rt| async move {
846
            let vanguardmgr = VanguardMgr::new_testing(&rt, VanguardMode::Full).unwrap();
847

            
848
            let netdir = testnet::construct_netdir().unwrap_if_sufficient().unwrap();
849
            let params = VanguardParams::try_from(netdir.params()).unwrap();
850
            let mut rng = testing_rng();
851

            
852
            // The sets are initially empty
853
            assert_sets_empty(&vanguardmgr);
854

            
855
            // Wait until the vanguard manager has bootstrapped
856
            let _netdir_provider = vanguardmgr.init_vanguard_sets(&netdir).await.unwrap();
857

            
858
            assert_sets_filled(&vanguardmgr, &params);
859

            
860
            let vanguard1 = vanguardmgr
861
                .select_vanguard(&mut rng, &netdir, Layer2, &permissive_selector())
862
                .unwrap();
863
            assert_expiry_in_bounds(&vanguard1, &vanguardmgr, &rt, &params, Layer2);
864

            
865
            let exclusion = RelayExclusion::exclude_identities(
866
                vanguard1
867
                    .relay()
868
                    .identities()
869
                    .map(|id| id.to_owned())
870
                    .collect(),
871
            );
872
            let selector =
873
                RelaySelector::new(tor_relay_selection::RelayUsage::vanguard(), exclusion);
874

            
875
            let vanguard2 = vanguardmgr
876
                .select_vanguard(&mut rng, &netdir, Layer3, &selector)
877
                .unwrap();
878

            
879
            assert_expiry_in_bounds(&vanguard2, &vanguardmgr, &rt, &params, Layer3);
880
            // Ensure we didn't select the same vanguard twice
881
            assert_ne!(
882
                vanguard1.relay().identities().collect_vec(),
883
                vanguard2.relay().identities().collect_vec()
884
            );
885
        });
886
    }
887

            
888
    /// Override the vanguard params from the netdir, returning the new VanguardParams.
889
    ///
890
    /// This also waits until the vanguard manager has had a chance to process the changes.
891
    async fn install_new_params(
892
        rt: &MockRuntime,
893
        netdir_provider: &TestNetDirProvider,
894
        params: impl IntoIterator<Item = (&str, i32)>,
895
    ) -> VanguardParams {
896
        let new_netdir = testnet::construct_custom_netdir_with_params(|_, _, _| {}, params, None)
897
            .unwrap()
898
            .unwrap_if_sufficient()
899
            .unwrap();
900
        let new_params = VanguardParams::try_from(new_netdir.params()).unwrap();
901

            
902
        netdir_provider.set_netdir_and_notify(new_netdir).await;
903

            
904
        // Wait until the vanguard mgr has finished handling the new netdir.
905
        rt.progress_until_stalled().await;
906

            
907
        new_params
908
    }
909

            
910
    /// Switch the vanguard "mode" of the VanguardMgr to `mode`,
911
    /// by setting the vanguards-hs-service parameter.
912
    //
913
    // TODO(#1382): use this instead of switch_hs_mode_config.
914
    #[allow(unused)]
915
    async fn switch_hs_mode(
916
        rt: &MockRuntime,
917
        vanguardmgr: &VanguardMgr<MockRuntime>,
918
        netdir_provider: &TestNetDirProvider,
919
        mode: VanguardMode,
920
    ) {
921
        use VanguardMode::*;
922

            
923
        let _params = match mode {
924
            Lite => install_new_params(rt, netdir_provider, ENABLE_LITE_VANGUARDS).await,
925
            Full => install_new_params(rt, netdir_provider, ENABLE_FULL_VANGUARDS).await,
926
            Disabled => panic!("cannot disable vanguards in the vanguard tests!"),
927
        };
928

            
929
        assert_eq!(vanguardmgr.mode(), mode);
930
    }
931

            
932
    /// Switch the vanguard "mode" of the VanguardMgr to `mode`,
933
    /// by calling `VanguardMgr::reconfigure`.
934
    fn switch_hs_mode_config(vanguardmgr: &VanguardMgr<MockRuntime>, mode: VanguardMode) {
935
        let _ = vanguardmgr
936
            .reconfigure(&VanguardConfig {
937
                mode: ExplicitOrAuto::Explicit(mode),
938
            })
939
            .unwrap();
940

            
941
        assert_eq!(vanguardmgr.mode(), mode);
942
    }
943

            
944
    /// Use a new NetDir that excludes one of our L2 vanguards
945
    async fn install_netdir_excluding_vanguard<'a>(
946
        runtime: &MockRuntime,
947
        vanguard: &Vanguard<'_>,
948
        params: impl IntoIterator<Item = (&'a str, i32)>,
949
        netdir_provider: &TestNetDirProvider,
950
    ) -> NetDir {
951
        let new_netdir = construct_custom_netdir_with_params(
952
            |_idx, bld, _| {
953
                let md_so_far = bld.md.testing_md().unwrap();
954
                if md_so_far.ed25519_id() == vanguard.relay().id() {
955
                    bld.omit_rs = true;
956
                }
957
            },
958
            params,
959
            None,
960
        )
961
        .unwrap()
962
        .unwrap_if_sufficient()
963
        .unwrap();
964

            
965
        netdir_provider
966
            .set_netdir_and_notify(new_netdir.clone())
967
            .await;
968
        // Wait until the vanguard mgr has finished handling the new netdir.
969
        runtime.progress_until_stalled().await;
970

            
971
        new_netdir
972
    }
973

            
974
    #[test]
975
    fn override_vanguard_set_size() {
976
        MockRuntime::test_with_various(|rt| async move {
977
            let vanguardmgr = VanguardMgr::new_testing(&rt, VanguardMode::Lite).unwrap();
978
            let netdir = testnet::construct_netdir().unwrap_if_sufficient().unwrap();
979
            // Wait until the vanguard manager has bootstrapped
980
            let netdir_provider = vanguardmgr.init_vanguard_sets(&netdir).await.unwrap();
981

            
982
            let params = VanguardParams::try_from(netdir.params()).unwrap();
983
            let old_size = params.l2_pool_size();
984
            assert_set_vanguards_targets_match_params(&vanguardmgr, &params);
985

            
986
            const PARAMS: [[(&str, i32); 2]; 2] = [
987
                [("guard-hs-l2-number", 1), ("guard-hs-l3-number", 10)],
988
                [("guard-hs-l2-number", 10), ("guard-hs-l3-number", 10)],
989
            ];
990

            
991
            for params in PARAMS {
992
                let new_params = install_new_params(&rt, &netdir_provider, params).await;
993

            
994
                // Ensure the target size was updated.
995
                assert_set_vanguards_targets_match_params(&vanguardmgr, &new_params);
996
                {
997
                    let inner = vanguardmgr.inner.read().unwrap();
998
                    let l2_vanguards = inner.l2_vanguards();
999
                    let l3_vanguards = inner.l3_vanguards();
                    let new_l2_size = params[0].1 as usize;
                    if new_l2_size < old_size {
                        // The actual size of the set hasn't changed: it's OK to have more vanguards than
                        // needed in the set (they extraneous ones will eventually expire).
                        assert_eq!(l2_vanguards.len(), old_size);
                    } else {
                        // The new size is greater, so we have more L2 vanguards now.
                        assert_eq!(l2_vanguards.len(), new_l2_size);
                    }
                    // There are no L3 vanguards because full vanguards are not in use.
                    assert_eq!(l3_vanguards.len(), 0);
                }
            }
        });
    }
    #[test]
    fn expire_vanguards() {
        MockRuntime::test_with_various(|rt| async move {
            let vanguardmgr = VanguardMgr::new_testing(&rt, VanguardMode::Lite).unwrap();
            let netdir = testnet::construct_netdir().unwrap_if_sufficient().unwrap();
            let params = VanguardParams::try_from(netdir.params()).unwrap();
            let initial_l2_number = params.l2_pool_size();
            // Wait until the vanguard manager has bootstrapped
            let netdir_provider = vanguardmgr.init_vanguard_sets(&netdir).await.unwrap();
            assert_eq!(vanguard_count(&vanguardmgr), params.l2_pool_size());
            // Find the RelayIds of the vanguard that is due to expire next
            let vanguard_id = {
                let inner = vanguardmgr.inner.read().unwrap();
                let next_expiry = inner.vanguard_sets.next_expiry().unwrap();
                inner
                    .l2_vanguards()
                    .iter()
                    .find(|v| v.when == next_expiry)
                    .cloned()
                    .unwrap()
                    .id
            };
            const FEWER_VANGUARDS_PARAM: [(&str, i32); 1] = [("guard-hs-l2-number", 1)];
            // Set the number of L2 vanguards to a lower value to ensure the vanguard that is about
            // to expire is not replaced. This allows us to test that it has indeed expired
            // (we can't simply check that the relay is no longer is the set,
            // because it's possible for the set to get replenished with the same relay).
            let new_params = install_new_params(&rt, &netdir_provider, FEWER_VANGUARDS_PARAM).await;
            // The vanguard has not expired yet.
            let timebound_vanguard = find_in_set(&vanguard_id, &vanguardmgr, Layer2);
            assert!(timebound_vanguard.is_some());
            assert_eq!(vanguard_count(&vanguardmgr), initial_l2_number);
            let lifetime = duration_until_expiry(&vanguard_id, &vanguardmgr, &rt, Layer2);
            // Wait until this vanguard expires
            rt.advance_by(lifetime).await.unwrap();
            rt.progress_until_stalled().await;
            let timebound_vanguard = find_in_set(&vanguard_id, &vanguardmgr, Layer2);
            // The vanguard expired, but was not replaced.
            assert!(timebound_vanguard.is_none());
            assert_eq!(vanguard_count(&vanguardmgr), initial_l2_number - 1);
            // Wait until more vanguards expire. This will reduce the set size to 1
            // (the new target size we set by overriding the params).
            for _ in 0..initial_l2_number - 1 {
                let vanguard_id = {
                    let inner = vanguardmgr.inner.read().unwrap();
                    let next_expiry = inner.vanguard_sets.next_expiry().unwrap();
                    inner
                        .l2_vanguards()
                        .iter()
                        .find(|v| v.when == next_expiry)
                        .cloned()
                        .unwrap()
                        .id
                };
                let lifetime = duration_until_expiry(&vanguard_id, &vanguardmgr, &rt, Layer2);
                rt.advance_by(lifetime).await.unwrap();
                rt.progress_until_stalled().await;
            }
            assert_eq!(vanguard_count(&vanguardmgr), new_params.l2_pool_size());
            // Update the L2 set size again, to force the vanguard manager to replenish the L2 set.
            const MORE_VANGUARDS_PARAM: [(&str, i32); 1] = [("guard-hs-l2-number", 5)];
            // Set the number of L2 vanguards to a lower value to ensure the vanguard that is about
            // to expire is not replaced. This allows us to test that it has indeed expired
            // (we can't simply check that the relay is no longer is the set,
            // because it's possible for the set to get replenished with the same relay).
            let new_params = install_new_params(&rt, &netdir_provider, MORE_VANGUARDS_PARAM).await;
            // Check that we replaced the expired vanguard with a new one:
            assert_eq!(vanguard_count(&vanguardmgr), new_params.l2_pool_size());
            {
                let inner = vanguardmgr.inner.read().unwrap();
                let l2_count = inner.l2_vanguards().len();
                assert_eq!(l2_count, new_params.l2_pool_size());
            }
        });
    }
    #[test]
    fn full_vanguards_persistence() {
        MockRuntime::test_with_various(|rt| async move {
            let vanguardmgr = VanguardMgr::new_testing(&rt, VanguardMode::Lite).unwrap();
            let netdir =
                construct_custom_netdir_with_params(|_, _, _| {}, ENABLE_LITE_VANGUARDS, None)
                    .unwrap()
                    .unwrap_if_sufficient()
                    .unwrap();
            let netdir_provider = vanguardmgr.init_vanguard_sets(&netdir).await.unwrap();
            // Full vanguards are not enabled, so we don't expect anything to be written
            // to persistent storage.
            assert_eq!(vanguardmgr.mode(), VanguardMode::Lite);
            assert!(vanguardmgr.storage.load().unwrap().is_none());
            let mut rng = testing_rng();
            assert!(
                vanguardmgr
                    .select_vanguard(&mut rng, &netdir, Layer3, &permissive_selector())
                    .is_err()
            );
            // Enable full vanguards again.
            //
            // We expect VanguardMgr to populate the L3 set, and write the VanguardSets to storage.
            switch_hs_mode_config(&vanguardmgr, VanguardMode::Full);
            rt.progress_until_stalled().await;
            let vanguard_sets_orig = vanguardmgr.storage.load().unwrap();
            assert!(
                vanguardmgr
                    .select_vanguard(&mut rng, &netdir, Layer3, &permissive_selector())
                    .is_ok()
            );
            // Switch to lite vanguards.
            switch_hs_mode_config(&vanguardmgr, VanguardMode::Lite);
            // The vanguard sets should not change when switching between lite and full vanguards.
            assert_eq!(vanguard_sets_orig, vanguardmgr.storage.load().unwrap());
            switch_hs_mode_config(&vanguardmgr, VanguardMode::Full);
            assert_eq!(vanguard_sets_orig, vanguardmgr.storage.load().unwrap());
            // TODO HS-VANGUARDS: we may want to disable the ability to switch back to lite
            // vanguards.
            // Switch to lite vanguards and remove a relay from the consensus.
            // The relay should *not* be persisted to storage until we switch back to full
            // vanguards.
            switch_hs_mode_config(&vanguardmgr, VanguardMode::Lite);
            let mut rng = testing_rng();
            let excluded_vanguard = vanguardmgr
                .select_vanguard(&mut rng, &netdir, Layer2, &permissive_selector())
                .unwrap();
            let _ = install_netdir_excluding_vanguard(
                &rt,
                &excluded_vanguard,
                ENABLE_LITE_VANGUARDS,
                &netdir_provider,
            )
            .await;
            // The vanguard sets from storage haven't changed, because we are in "lite" mode.
            assert_eq!(vanguard_sets_orig, vanguardmgr.storage.load().unwrap());
            let _ = install_netdir_excluding_vanguard(
                &rt,
                &excluded_vanguard,
                ENABLE_FULL_VANGUARDS,
                &netdir_provider,
            )
            .await;
        });
    }
    #[test]
    fn load_from_state_file() {
        MockRuntime::test_with_various(|rt| async move {
            // Set the wallclock to a time when some of the stored vanguards are still valid.
            let now = time::UNIX_EPOCH + Duration::from_secs(1610000000);
            rt.jump_wallclock(now);
            let config = VanguardConfig {
                mode: ExplicitOrAuto::Explicit(VanguardMode::Full),
            };
            // The state file contains no vanguards
            let (statemgr, _dir) =
                state_dir_with_vanguards(r#"{ "l2_vanguards": [], "l3_vanguards": [] }"#);
            let vanguardmgr = VanguardMgr::new(&config, rt.clone(), statemgr, false).unwrap();
            {
                let inner = vanguardmgr.inner.read().unwrap();
                // The vanguard sets should be empty too
                assert!(inner.vanguard_sets.l2().is_empty());
                assert!(inner.vanguard_sets.l3().is_empty());
            }
            let (statemgr, _dir) = state_dir_with_vanguards(VANGUARDS_JSON);
            let vanguardmgr =
                Arc::new(VanguardMgr::new(&config, rt.clone(), statemgr, false).unwrap());
            let (initial_l2, initial_l3) = {
                let inner = vanguardmgr.inner.read().unwrap();
                let l2_vanguards = inner.vanguard_sets.l2_vanguards().clone();
                let l3_vanguards = inner.vanguard_sets.l3_vanguards().clone();
                // The sets actually contain 4 and 5 vanguards, respectively,
                // but the expired ones are discarded.
                assert_eq!(l2_vanguards.len(), 3);
                assert_eq!(l3_vanguards.len(), 2);
                // We don't know how many vanguards we're going to need
                // until we fetch the consensus.
                assert_eq!(inner.vanguard_sets.l2_vanguards_target(), 0);
                assert_eq!(inner.vanguard_sets.l3_vanguards_target(), 0);
                assert_eq!(inner.vanguard_sets.l2_vanguards_deficit(), 0);
                assert_eq!(inner.vanguard_sets.l3_vanguards_deficit(), 0);
                (l2_vanguards, l3_vanguards)
            };
            let netdir = testnet::construct_netdir().unwrap_if_sufficient().unwrap();
            let _netdir_provider = vanguardmgr.init_vanguard_sets(&netdir).await.unwrap();
            {
                let inner = vanguardmgr.inner.read().unwrap();
                let l2_vanguards = inner.vanguard_sets.l2_vanguards();
                let l3_vanguards = inner.vanguard_sets.l3_vanguards();
                // The sets were replenished with more vanguards
                assert_eq!(l2_vanguards.len(), 4);
                assert_eq!(l3_vanguards.len(), 8);
                // We now know we need 4 L2 vanguards and 8 L3 ones.
                assert_eq!(inner.vanguard_sets.l2_vanguards_target(), 4);
                assert_eq!(inner.vanguard_sets.l3_vanguards_target(), 8);
                assert_eq!(inner.vanguard_sets.l2_vanguards_deficit(), 0);
                assert_eq!(inner.vanguard_sets.l3_vanguards_deficit(), 0);
                // All of the vanguards read from the state file should still be in the sets.
                assert!(initial_l2.iter().all(|v| l2_vanguards.contains(v)));
                assert!(initial_l3.iter().all(|v| l3_vanguards.contains(v)));
            }
        });
    }
    #[test]
    fn invalid_state_file() {
        MockRuntime::test_with_various(|rt| async move {
            let config = VanguardConfig {
                mode: ExplicitOrAuto::Explicit(VanguardMode::Full),
            };
            let (statemgr, _dir) = state_dir_with_vanguards(INVALID_VANGUARDS_JSON);
            let res = VanguardMgr::new(&config, rt.clone(), statemgr, false);
            assert!(matches!(res, Err(VanguardMgrError::State(_))));
        });
    }
}