mirrors/arti.git/crates/tor-circmgr/src/impls.rs, branch arti-v1.4.6

tor-proto: Update the TunnelMutableState when a circuit is removed.

2025-05-15T16:33:33Z

This is messy, because `ClientCirc::{path_ref, n_hops, ..}` become fallible (we can't unwrap the result, because when a circuit is closed, its state gets removed from the `TunnelSharedState`, but its `ClientCirc` handle continues to exist, so any attempt to retrieve the state will result in an `Err`).

circmgr: Rename AbstractCirc::{extend_ntor => extend}

2025-04-28T17:29:02Z

We don't want to be thinking about ntor vs ntor3 in circmgr.

tor-proto: New extend() to pick between ntor and ntor3

2025-04-28T17:24:11Z

In the future, when we add more circuit handshakes (PQ anyone?) we'll want to have the logic for choosing which to use be unified. Almost nobody calling tor-proto should need to care which circuit handshake is going to be used. Closes #1970.

circ: Remove CircParameters reference in call stack

2025-04-23T18:36:28Z

This avoids cloning the object and instead allows us to have a CircParameters per hop on the circuit path. This will come handy with congestion control where each hop might have different congestion control parameters. Part of #1817 Signed-off-by: David Goulet

squash! Upgrade rand dependency to 0.9.

2025-03-18T16:09:44Z

- `rand::thread_rng()` has been deprecated and renamed to `rand::rng()`

circmgr: Remove the CircParameters build .expect()

2025-01-16T14:16:01Z

Instead, return an error and make all call site handle it. Signed-off-by: David Goulet

circ: Specialize the circparams from netparams function

2025-01-16T14:16:01Z

Congestion control parameters have specific values depending on the circuit type. Instead of using a CircuitType, which is removed in this commit, specialize the function in this case onion and exit. This allows us to get rid of CircuitType and solely use TargetCircUsage instead. At this commit, we use .expect() on the Builder. Future commit will remove this to return a Result in case of failure. Worth noting that we don't expect one. Signed-off-by: David Goulet

circmgr: Modify CircParameters for congestion control

2025-01-16T14:16:01Z

The congestion control parameters are created from the consensus parameters (netparams) and then put into the CircParameters object that is then passed down the tor-proto crate. Because different parameters are selected depending on the circuit type (onion vs exit vs sbws), a CircuitType enum is introduced for the sole purpose of being used to select the right parameters. Related #534 Signed-off-by: David Goulet

CircMgr: Start writing tests.

2024-10-03T12:56:16Z

CircMgr: Make CircMgr and HsCircPool generic over builder type.

2024-09-24T13:45:21Z

This will allow for testing, as the CircuitBuilder can be replaced with a mocked version. This did require moving some of what was in the CircuitBuilder impl into the AbstractCircuitBuilder type, since Drop implementations can't be specialized, but that's fine, as we'll probably be doing more of that in the future anyways.