mirrors/arti.git/crates/tor-proto/src/client/reactor, branch arti-v2.0.0mirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=arti-v2.0.02026-01-29T16:33:21Zproto: Avoid locking in CircHopOutbound::ccontrol()2026-01-29T16:33:21ZGabriela Moldovangabi@torproject.org2026-01-22T11:54:26Zurn:sha1:29159e80c713ea4ab1ab657c7eeae53f37e62814
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<Mutex<..>>`,
but I'd prefer keeping the API small.
proto: Rip CC state out of CircHopInbound2026-01-29T16:33:21ZGabriela Moldovangabi@torproject.org2026-01-16T16:51:17Zurn:sha1:ce6c90cca94b02e6d78d7cca7f01ec4f675f4c28
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: Move padding_disposition() to the shared padding module2026-01-29T16:33:21ZGabriela Moldovangabi@torproject.org2026-01-23T14:40:04Zurn:sha1:67c35a98388f6eaeea852e44d8a858ebf815a59fproto: Move CircPaddingDisposition to a new module2026-01-29T16:33:21ZGabriela Moldovangabi@torproject.org2026-01-23T14:30:15Zurn:sha1:93db46cc7e5124e2211e54c43652fafebcddae35
Relays will need to use it too.
proto: Move TimeoutEstimator to util2026-01-29T16:33:21ZGabriela Moldovangabi@torproject.org2026-01-12T10:44:05Zurn:sha1:5c5bf40652f00fff3205fc5a2ff44ad48e96b8de
This will be used in the stream reactor too (and the stream reactor will
eventually replace the corresponding client impl).
proto: Dedupe IncomingStreamRequestHandler2026-01-12T19:10:54ZGabriela Moldovangabi@torproject.org2026-01-12T18:50:26Zurn:sha1:271603e38f5be23ae3ff756a18fb87406a933e9dproto: Rename CircuitAction and cie to CircuitEvent2025-12-10T16:04:26ZDavid Gouletdgoulet@torproject.org2025-12-10T15:20:41Zurn:sha1:7e69f0b2d31acfad14c3d046e89f8633626785f8
This mirrors also the relay reactor. We've introduced the ProtoViolation
into a previous commit which is not an action but rather an "event" that
happened on a circuit.
And so, better semantic. No behavior change.
Signed-off-by: David Goulet <dgoulet@torproject.org>
proto: Rename Circuit{Action/Event}::Shutdown to ProtoViolation2025-12-10T16:04:26ZDavid Gouletdgoulet@torproject.org2025-12-10T14:24:59Zurn:sha1:1c97ec8836f227209e8dbc19253afcbe1f9e2610
Signed-off-by: David Goulet <dgoulet@torproject.org>
proto: Add TODO in client reactor for bad cell2025-12-10T16:04:26ZDavid Gouletdgoulet@torproject.org2025-12-09T15:06:26Zurn:sha1:d83f860282020e535a50ebb183fe27e41b2d5830
Because of https://gitlab.torproject.org/tpo/core/torspec/-/issues/385
Signed-off-by: David Goulet <dgoulet@torproject.org>
proto: Client circuit reactor now handles AnyChanMsg2025-12-10T16:03:12ZDavid Gouletdgoulet@torproject.org2025-12-08T19:16:33Zurn:sha1:d5ad6142db84d917f0035060f92678da3ba98ee1
This commit removes the CircuitRx* based solely on the client circuit
message and moves it into the top level of the crate so all reactors can
use them.
The client reactor then upon receiving the message, it converts the
AnyChanMsg into a ClientCircChanMsg. On error, this leads to a shutdown
of the entire reactor due to a fatal error.
In order to pull this off, we added a CircuitAction::Shutdown that is
handled as a priority.
Signed-off-by: David Goulet <dgoulet@torproject.org>