| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | | conflux: Add test for out-of-order cell handling in the client. | Gabriela Moldovan | 2025-07-16 | 1 | -3/+252 | |
| | | | ||||||
| * | | proto: Make the mock exit start writing to the stream. | Gabriela Moldovan | 2025-07-16 | 1 | -1/+84 | |
| | | | ||||||
| * | | proto: Replace oneshot completion channel with event channel. | Gabriela Moldovan | 2025-07-16 | 1 | -18/+45 | |
| | | | | | | | | | | | | | | | The two mock exit legs will use this channel to notify each other of the receipt of BEGIN cells. This will enable us to extend the tests to support writing cells to the stream (but only after the stream is opened). | |||||
| * | | proto: Give ConfluxStreamState a constructor. | Gabriela Moldovan | 2025-07-16 | 1 | -7/+14 | |
| | | | ||||||
| * | | proto: Replace tuple with struct with named fields for clarity. | Gabriela Moldovan | 2025-07-16 | 1 | -37/+30 | |
| | | | | | | | | | This abolishes the overly-long tuple used in these tests. | |||||
| * | | proto: Move list of expected switches to a separate variable. | Gabriela Moldovan | 2025-07-16 | 1 | -26/+30 | |
| | | | | | | | | | | | I am moving these out of the `relays` list, in preparation for a minor refactoring around that part. | |||||
| * | | proto: Clarify docs to say who the receiver of the data_recvd is. | Gabriela Moldovan | 2025-07-16 | 1 | -1/+1 | |
| | | | ||||||
| * | | proto: Remove no-longer accurate comment from test. | Gabriela Moldovan | 2025-07-16 | 1 | -4/+0 | |
| | | | ||||||
| * | | proto: Refactor test loop exit condition. | Gabriela Moldovan | 2025-07-16 | 1 | -1/+13 | |
| | | | | | | | | | | | This is about to become slightly more complex, as the mock exit will support writing to the stream too. | |||||
| * | | proto: Rename test for clarity. | Gabriela Moldovan | 2025-07-16 | 1 | -1/+1 | |
| |/ | | | | | I will another test for exit-to-client transmissions, so I am preemptively renaming its counterpart. | |||||
| * | Merge branch 'flow-ctrl' into 'main' | opara | 2025-07-10 | 12 | -96/+327 | |
| |\ | | | | | | | | | tor-proto: Handle incoming XON/XOFF messages See merge request tpo/core/arti!3054 | |||||
| | * | tor-proto: renamed `get_bytes_per_sec()` to `bytes_per_sec()` | Steven Engler | 2025-07-10 | 2 | -2/+2 | |
| | | | ||||||
| | * | tor-proto: changed 'if' statements to 'match' | Steven Engler | 2025-07-10 | 2 | -22/+34 | |
| | | | ||||||
| | * | tor-proto: added TODOs for making rate limit stream optional | Steven Engler | 2025-07-09 | 2 | -0/+2 | |
| | | | ||||||
| | * | tor-proto: set `XonXoffBased` variant behind "flowctl-cc" feat | Steven Engler | 2025-07-07 | 2 | -5/+23 | |
| | | | ||||||
| | * | tor-proto: finish support for `StreamSendFlowControl` XON/XOFF | Steven Engler | 2025-07-07 | 1 | -7/+4 | |
| | | | ||||||
| | * | tor-proto: handle XON/XOFF messages | Steven Engler | 2025-07-07 | 5 | -7/+101 | |
| | | | ||||||
| | * | tor-proto: establish channel for rate limit updates | Steven Engler | 2025-07-07 | 9 | -31/+127 | |
| | | | ||||||
| | * | tor-proto: code movement | Steven Engler | 2025-07-07 | 1 | -33/+33 | |
| | | | | | | | | | Moves the `DataWriter` impl to immediately after the definition. | |||||
| | * | tor-proto: small `DataWriter` refactoring | Steven Engler | 2025-07-07 | 1 | -25/+35 | |
| | | | ||||||
| | * | tor-proto: remove incorrect doc comment | Steven Engler | 2025-07-07 | 1 | -2/+0 | |
| | | | ||||||
| | * | tor-proto: move sendme decoding into `StreamSendFlowControl` | Steven Engler | 2025-07-07 | 4 | -18/+22 | |
| | | | | | | | | | | | We only want to try to decode it if the circuit is using window-based flow control. | |||||
| * | | Merge branch 'fix-zero-read' into 'main' | Ian Jackson | 2025-07-10 | 1 | -2/+3 | |
| |\ \ | | | | | | | | | | | | | | | | | | | tor-proto: Fix 0-len buffer bug in `DataReader::poll_read` Closes #2053 See merge request tpo/core/arti!3080 | |||||
| | * | | tor-proto: fix 0-len buffer bug in `DataReader::poll_read` | Steven Engler | 2025-06-26 | 1 | -2/+3 | |
| | | | | ||||||
| * | | | Merge branch 'prop360-combined' into 'main' | Nick Mathewson | 2025-07-10 | 4 | -1/+144 | |
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | Implement proposal 360 (limit HsDesc inflation opportunities) Closes #2046 See merge request tpo/core/arti!3070 | |||||
| | * | | | Suggestions from @gabi-250 on prop360 implementation. | Nick Mathewson | 2025-07-10 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | Comment to explain _why_ ExcessOutbound is internal. | Nick Mathewson | 2025-07-10 | 1 | -0/+5 | |
| | | | | | ||||||
| | * | | | Rename last_hop to last_hop_info to fix name conflict | Nick Mathewson | 2025-07-10 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | proto: Add the ability to enforce a maximum number of cells per hop | Nick Mathewson | 2025-07-10 | 4 | -1/+118 | |
| | | | | | ||||||
| | * | | | proto: Add method to get the last hop of a circuit. | Nick Mathewson | 2025-07-10 | 1 | -0/+21 | |
| | | |/ | |/| | ||||||
| * / | | Typo fixes (automatic and hand-verified) | Nick Mathewson | 2025-07-09 | 2 | -2/+2 | |
| |/ / | | | | | | | Made with https://crates.io/crates/typos-cli | |||||
| * | | tor-proto: Remove unnecessary return. | Gabriela Moldovan | 2025-07-07 | 1 | -1/+1 | |
| | | | ||||||
| * | | Temporarily suppress mismatched_lifetime_syntaxes. | Gabriela Moldovan | 2025-07-07 | 1 | -0/+1 | |
| | | | | | | | | | See #2060. | |||||
| * | | proto: ignore large_enum_variant warning | Nick Mathewson | 2025-07-03 | 1 | -1/+1 | |
| | | | | | | | | | | | This appears to be a testing-only struct, so we can safely ignore the warning. | |||||
| * | | proto: Add clippy exception for handle_cmd(). | Gabriela Moldovan | 2025-06-27 | 1 | -0/+1 | |
| | | | | | | | | | We need to pass by value for the conflux case. | |||||
| * | | proto: Add clippy exception for next_circ_action(). | Gabriela Moldovan | 2025-06-27 | 1 | -0/+1 | |
| | | | | | | | | | The function can return an error, but only if `conflux` is enabled. | |||||
| * | | proto: Remove unnecessary return statement to satisfy clippy. | Gabriela Moldovan | 2025-06-27 | 1 | -1/+1 | |
| | | | | | | | | | With the latest rustc version, this now triggers a warning. | |||||
| * | | proto: Use primary_join_point() earlier to get the join point. | Gabriela Moldovan | 2025-06-27 | 1 | -3/+2 | |
| | | | ||||||
| * | | proto: Rephrase error message to reduce ambiguity. | Gabriela Moldovan | 2025-06-27 | 1 | -1/+1 | |
| | | | ||||||
| * | | proto: Continue reading from leaky-pipe streams if main leg is blocked (fmt). | Gabriela Moldovan | 2025-06-27 | 3 | -10/+21 | |
| | | | ||||||
| * | | proto: Continue reading from leaky-pipe streams if main leg is blocked. | Gabriela Moldovan | 2025-06-27 | 4 | -90/+118 | |
| | | | | | | | | | Addresses https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3077#note_3218281 | |||||
| * | | proto: Allow leaky pipe on multipath tunnels. | Gabriela Moldovan | 2025-06-27 | 1 | -20/+17 | |
| | | | | | | | | | | | | | | | | | For leaky pipe, we just need to ensure we don't accidentally reroute it to the join point on the primary leg. Partially addresses https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3077#note_3218281 | |||||
| * | | proto: Actually validate the SWITCH cells in the mock exit. | Gabriela Moldovan | 2025-06-27 | 1 | -12/+43 | |
| | | | ||||||
| * | | proto: Tweak conflux test RTTs to trigger CONFLUX_SWITCH. | Gabriela Moldovan | 2025-06-27 | 1 | -6/+49 | |
| | | | | | | | | | This causes the test client to send a two SWITCH cells. | |||||
| * | | proto: Make RttEstimator::{ewma,min}_rtt_usec() return `Option`. | Gabriela Moldovan | 2025-06-27 | 3 | -13/+20 | |
| | | | | | | | | | | | | | | | | | | | | | A `None` value signals to the conflux code to know to fall back on the initial RTT of the circuit. Without this change, the conflux switching logic is broken as we end up staying on the leg with the best initial RTT forever (the other leg is never picked, because its `ewma_rtt()` is stuck on `u32::MAX`, and never updated as we never send on it). | |||||
| * | | proto: Ugly hack to fix seqno accounting bug. | Gabriela Moldovan | 2025-06-27 | 4 | -0/+31 | |
| | | | ||||||
| * | | proto: Run test handshake outside of exit task. | Gabriela Moldovan | 2025-06-27 | 1 | -10/+17 | |
| | | | ||||||
| * | | proto: Avoid switching legs unnecssarily. | Gabriela Moldovan | 2025-06-27 | 1 | -1/+1 | |
| | | | | | | | | | | | If we're already on the best leg, we don't need to switch (even if the other leg happens to have the same "best" RTT). | |||||
| * | | proto: Ensure we get all the expected SWITCH cells. | Gabriela Moldovan | 2025-06-27 | 1 | -0/+6 | |
| | | | | | | | | | | | | | | | | | | | Previously, this failed to assert that the mock relays received all the expected SWITCH cells. As it turns out, the assertion currently fails, because the default value for our estimated RTTs is no longer zero (as of !3074), so we no longer unnecessarily switch legs as much as we used to. | |||||
| * | | proto: Sort the received test data. | Gabriela Moldovan | 2025-06-27 | 1 | -4/+4 | |
| | | | | | | | | | | | | | | | | | | | | | The conflux client now sends data to the mock exit relay over both legs (it SWITCHes legs at some point), which causes the test to fail (because the two legs of the mock exit write the data racily to the same `Vec`, without attempting to put it in the right order). To work around the lack of handling of out-of-order cells at the mock exit, we can just sort the received data to make sure we got it all. | |||||
