mirrors/arti.git/crates/tor-proto/src/circuit/reactor.rs, branch arti-v1.2.6

circuit reactor: add comment pointing to tests in other module

2024-07-29T16:15:06Z

Fix two TODO ticket numbers

2024-06-25T10:56:14Z

I seem to have mistyped this ticket number. We meant #1397 aka "Circuit reactor isn't great", not some release ticket.

Merge branch 'sometimes-unbounded' into 'main'

2024-06-11T20:35:38Z

RFC: Provide and use SometimesUnboundedSender in circuit reactor See merge request tpo/core/arti!2172

Merge branch 'poll-ready-unpin-bool' into 'main'

2024-05-29T18:05:57Z

Tidy up the ChannelSender::poll_ready inherent method See merge request tpo/core/arti!2171

Use SometimesUnboundedSink for the circuit reactor's bag-on-the-side

2024-05-29T13:15:25Z

proto: Make DataWriter::close actually do something.

2024-05-29T11:28:01Z

Previously we had a bug where `::close` (or `shutdown` in tokio-land) would not actually have any effect. It _would_ drop the `StreamTarget` held by the `DataWriter`, but since the `DataReader` also held a `StreamTarget`, the MPSC channel would not get closed, and the circuit reactor would not realize that the stream wanted to shut down. Now we use `mpsc::Sender::close_channel` to make our closes effectual. Closes #1368. Additionally, we fix a bug where `poll_close()` never actually did anything if the buffer had nothing in it when it was called. Previously, `poll_flush_impl()` would exit immediately if it had no data to flush. That isn't what we want when we are closing!

ChannelSender::poll_ready_unpin_bool: move to util

2024-05-29T11:16:19Z

This is where it belongs.

ChannelSender::poll_ready_unpin_bool: extension trait

2024-05-29T11:16:15Z

This makes this available for any Sink + Unpin. Which we want because we're about to wrap our ChannelSender in a Sink wrapper. It's in the wrong place now; we'll move it in a moment.

ChannelSender::poll_ready_unpin_bool: rename from poll_ready

2024-05-29T11:15:53Z

This would otherwise shadow the poll_ready method, which is confusing. Also this paves the way for making it available for any Sink + Unpin. Improve the docs somewhat to explain what this thing actually is.

proto: Make Channel explicitly Arc<.>

2024-05-16T16:24:12Z

Previously, Channel was a type that you could Clone that implicitly its state. Now, Channel always appears as an Arc. This change has several benefits: * It makes the relationship between Channel struct and the underlying channel more clear. * It enables Channel to participate in the RPC system, where everything has to be an Arc<.> * It enables us to have a Weak, if we ever want to. * It will let us move various members out of ChannelDetails. We did this change a while ago with ClientCirc.