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

proto: Update the tests to use the new CircuitRx{Receiver,Sender}s

2026-06-08T14:07:16Z

proto: Replace CircuitRx{Sender,Receiver} with new channel type

2026-06-08T14:07:16Z

This is needed for relays as part of #2490. Note that changing this type affects the client implementation too (i.e. clients will start prioritizing inbound DESTROY, discarding any queued data without forwarding it to their local streams). But that's okay, because it will generally only affect misbehaving clients, and clients unlucky enough to encounter a hibernating relay.

proto: Add a new channel -> circuit queue type

2026-06-08T14:07:16Z

This adds new a `CircuitRxSender`/`CircuitRxReceiver` queue type. The corresponding (`Sink`-link) sender and `Stream` implementations prioritize the delivery of `DESTROY` messages, which get delivered immediately, even if there are other messages queued in the underlying MPSC message queue. We are okay with the resulting data loss, because inbound DESTROY can be indicative of malicious activity on the circuit. We choose to err on the safe side, and free up the resources associated with such circuits as soon as possible. DESTROY messages are also sent by relays when they're about to hibernate, and by clients once they've decided to stop using a circuit. In the latter case, the lack of an `RELAY_COMMAND_END_ACK` does mean that this prioritization can cause data loss in cases where the client closes the circuit immediately after END-ing a stream. However, this is a deficiency in the protocol, and not something we want to fix by implementing custom flushing logic in the reactor. See torspec#196 and the discussion in #2490. Part of #2490

proto: Add a test helper function for building a CircNetParameters

2026-04-14T20:10:18Z

We can't access `NetParameters` in this crate to build from consensus default value so instead add a `defaults_for_tests()` gated function for each parameters object which puts in the defaults from the spec. We'll need CircNetParameters in order to build create request handler for unit tests hence why it is in src/circuit.rs.

proto: Rename CircSyncView to CircHopSyncView

2026-02-16T15:24:51Z

And update the docs

proto: Move CREATE helpers to a shared module

2026-02-05T16:04:59Z

These will be used by the relay code too (for circuit extension).

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: Add a new module for the generic circuit reactor

2026-01-29T16:33:21Z

Currently empty, will be fleshed out in a future commit.

proto; Factor HOPS constant out of maybenot_padding

2026-01-29T16:33:21Z

I am about to use this in other places too.

proto: Move CircPaddingDisposition to a new module

2026-01-29T16:33:21Z

Relays will need to use it too.