mirrors/arti.git/crates/tor-proto/src/stream, branch arti-v1.4.0

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.

proto: Clarify applicability of ClientStreamCtrl.

2025-01-30T15:30:54Z

proto: Use congestion control in circuit reactor

2025-01-16T14:16:01Z

It is official, congestion control is now used at this commit by the circuit reactor making circuit/sendme.rs unused. Will be removed with another commit. Related #534 Signed-off-by: David Goulet

memquota: Use _ rather than allow(dead_code) (fmt)

2024-10-22T12:19:22Z

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

memquota: fix data stream account lifetime

2024-10-21T10:11:20Z

The DataStream is sometimes disassembled, eg by split. When that happens, the StreamAccount would be dropped - and that was the only strong reference. Put a StreamAccount in each of the pieces, instead of just in the combined DataStream struct.

memquota: Fix resolve stream account lifetime

2024-10-21T10:11:20Z

We need the mq account for the stream not to collapse. The ResolveStream object needs to contain a strong reference to it. Have begin_stream_impl return the StreamAccount, rather than taking it as a parameter. That makes this bug a little more obvious. It also centralises the StreamAccount creation.