mirrors/arti.git/crates/tor-proto/src/stream/data.rs, branch arti-v1.4.4

cell, proto: Use correct Data sizes for v1 relay cells

2025-04-16T14:58:49Z

Since v1 cells have a longer tag, they can fit less data into a single cell. Ah well, that's the cost of improved security. The code in data.rs is a little wonky, in that it currently requires its buffer to be exactly the maximum size for a data cell. We have a TODO about fixing that in the future, but for now I've moved it to use a boxed slice rather than a boxed array. Part of #1944.

Apply 1 suggestion(s) to 1 file(s)

2025-02-27T13:24:05Z

Co-authored-by: Ian Jackson

Make DataStream, and its members, implement Sync.

2025-02-26T17:34:30Z

Also, use static_assertions to enforce that that they _stay_ Send+Sync. Closes #1859.

tor-proto: Add a tunnel module.

2025-02-20T20:15:12Z

Move StreamTarget to the tunnel module and the circuit module. From now on streams will be implemented on tunnels, not circuits. This moves `StreamTarget` to the tunnel module. A future change will replace `ClientCirc` with `ClientTunnel` inside `StreamTarget`. This is mostly code motion, best reviewed with `--color-moved`. Signed-off-by: David Goulet

tor-proto: Remove deprecated DataStream API.

2025-02-13T17:51:00Z

Reducing `ClientCirc` proliferation will make our lives easier when implementing !2790.

proto: deprecate DataStream::circuit

2025-01-30T16:11:01Z

This method is experimental, so no semver note is needed. It is redundant with `client_stream_ctrl()?.circuit()?`. (The name and the unconditional return type of this method are probably an error.)

proto: Rename (experimental) DataStream functions for ctrl access

2025-01-30T16:11:01Z

Since these return a client-specific type, they need a client-specific name before we can stabilize them for RPC.

proto: Rename ClientDataStreamCtrl::{is_open=>is_connected}

2025-01-30T16:11:01Z

Semantically, the new name matches the behavior much better. (A stream could well count as open if we had sent a RESOLVE but not received a RESOLVED, so we might someday want to have an `is_open` defined for _all_ streams, not just data streams.)

proto: Rename DataStreamCtrl to ClientDataStreamCtrl

2025-01-30T16:11:01Z

The API for this type, and the fact that it implements ClientStreamCtrl unconditionally, means that it is only for client DataStreams.

memquota: Use _ rather than allow(dead_code)

2024-10-22T12:19:22Z

Promote the associated comments. As suggested here: https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2560#note_3097188