mirrors/arti.git/crates/tor-proto/src/circuit, branch arti-v2.0.0

proto: Allow unused async in WIP reactor code

2026-01-29T16:33:24Z

This will need to become async soon.

proto: Document that the hop list is shared with the BWD

2026-01-29T16:33:21Z

proto: Rename the FWD -> BWD channel

2026-01-29T16:33:21Z

In the backward reactor, we call this the `forward_reactor_rx` (because it receives commands from the foward reactor), and in the forward reactor we call it `backward_reactor_tx` (because it sends commands to the backward reactor).

proto: Rename chan senders and sinks for clarity

2026-01-29T16:33:21Z

We settled on * `inbound_chan{tx, rx}`, for the inbound channel (the channel towards the guard, if we are a client, or towards the client if we are a relay) * `outbound_chan{tx, rx}`, for the outbound channel (the channel towards the exit, if we are a middle relay)

proto: Update misleading comment about cmd_rx

2026-01-29T16:33:21Z

proto: Allow CircSynvView::new_relay() to be unused

2026-01-29T16:33:21Z

This will change significantly in the near future, or disappear entirely.

proto: Add a new, implementation-agnostic circuit reactor

2026-01-29T16:33:21Z

proto: Avoid locking in CircHopOutbound::ccontrol()

2026-01-29T16:33:21Z

This is just because the generic reactor will soon need a clone of the CC object, so I am preemptively making this function return a ref to the underlying `Arc` instead. Technically, it would've been fine to just kept this method and add a separate one returning `&Arc>`, but I'd prefer keeping the API small.

proto: Rip CC state out of CircHopInbound

2026-01-29T16:33:21Z

Soon it won't need be needed here any more. I'm removing it, because having redundant handles to the CC state makes it difficult to see exactly where it's being used from.

proto: Factor ReactorResultChannel into the shared reactor module

2026-01-29T16:33:21Z

This is not just for clients!