summaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
...
* | | tor-proto: Fixup CircHop docs post-refactoring.Gabriela Moldovan2025-06-101-5/+6
| | |
* | | tor-proto: Remove now-addressed TODO.Gabriela Moldovan2025-06-101-4/+0
| | |
* | | tor-proto: Introduce CircHopList, hide CircHop internals.Gabriela Moldovan2025-06-103-244/+399
| | | | | | | | | | | | Most of this is code motion, I recommend reviewing with `--color-moved`.
* | | tor-proto: Move CircHop to a separate module.Gabriela Moldovan2025-06-102-163/+187
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This helps hide the `CircHop` internals, and is the first step towards providing a safer API that aims to reduce contention and prevent deadlocking on the stream map mutex. This change is also in preparation for implementing special handling for the join point of a conflux tunnel (which will involve adding a new `CircHop` API for sharing the stream map of another `CircHop`). I recommend reviewing this diff with `--color-moved`.
* | | Merge branch 'circparam_cleanup' into 'main'Nick Mathewson2025-06-1012-109/+214
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Initial refactoring on CircParameters Closes #1967 and #1968 See merge request tpo/core/arti!3021
| * | | HopSettings: Note relay usage.Nick Mathewson2025-06-101-0/+3
| | | |
| * | | Avoid a bug with HopSettingsNick Mathewson2025-06-102-5/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | When negotiation won't occur, we need to represent the fact by disabling any settings that would depend on negotiation. Otherwise we'll wind up with the client thinking everything was supported, and the relay thinking that nothing is.
| * | | Clarify HopSettings documentation.Nick Mathewson2025-06-101-3/+7
| | | |
| * | | Renamed NegotiatedHopSettings -> HopSettingsNick Mathewson2025-06-104-36/+33
| | | |
| * | | Clarify documentation for from_params_and_caps.Nick Mathewson2025-06-101-1/+6
| | | |
| * | | Clarify per-circuitness of CircParametersNick Mathewson2025-06-101-1/+7
| | | |
| * | | proto: Rename a pair of internal functionsNick Mathewson2025-06-103-16/+14
| | | |
| * | | Add a semver file for tor-proto changesNick Mathewson2025-06-101-0/+4
| | | |
| * | | proto: Refactor cc fallback.Nick Mathewson2025-06-104-19/+27
| | | | | | | | | | | | | | | | | | | | The fallback CC algorithm is _always_ fixed-window, and we should only use it when the selected CC algorithm is not supported.
| * | | Move responsibility for choosing extensions into tor-protoNick Mathewson2025-06-103-25/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now tor-circmgr no longer needs to check which Protover capabilities are enabled, or construct a separate CircParameters for each hop. Instead, tor-proto decides whether to use the fallback CC mode, based on whether the target supports FLOWCTRL_CC. Closes #1967.
| * | | proto: Move NegotiatedHopSettings to a higher levelNick Mathewson2025-06-108-61/+97
| | | | | | | | | | | | | | | | | | | | | | | | We will construct this object based on the circuit parameters _and_ on the target's supported protocol versions, so we need to do so when we have both pieces of info.
| * | | proto: Split CircParameters and NegotiatedHopSettingsNick Mathewson2025-06-106-32/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | One type will now represent _the kind of hop we are asking tor-proto to negotiate_; the other will represent _the state of such negotiation_. This doesn't simplify the code much yet, but it will be helpful as we add more and more negotiable settings. Part of #1967
* | | | Remove comment saying we do not implement consensus diffsNick Mathewson2025-06-101-2/+0
|/ / /
* / / Un-pin version of typed-index-collectionsNick Mathewson2025-06-091-3/+1
|/ / | | | | | | | | | | | | Now that our MSRV has been updated to 1.83, we should be able to get away with this. Closes #1647. Reverts !2471.
* | Merge branch 'fix-mockexecutor-spawn-blocking' into 'main'wesleyac2025-06-091-38/+6
|\ \ | | | | | | | | | | | | | | | | | | tor-rtmock: Use subthread_spawn for spawn_blocking. Closes #2033 See merge request tpo/core/arti!3036
| * | tor-rtmock: Use subthread_spawn for spawn_blocking.Wesley Aptekar-Cassels2025-06-091-38/+6
| | | | | | | | | | | | Fixes: #2033
* | | Merge branch 'criterion-fix-aarch64' into 'main'Nick Mathewson2025-06-099-19/+54
|\ \ \ | |/ / |/| | | | | | | | | | | | | | proto: Let benchmarks build correctly on non-intel Closes #2029 See merge request tpo/core/arti!3032
| * | Run cargo sort on tor-protoNick Mathewson2025-06-091-7/+15
| | |
| * | proto: Let benchmarks build correctly on non-intelNick Mathewson2025-06-099-26/+53
| |/ | | | | | | | | | | | | (criterion-cycles-per-byte, though quite helpful, doesn't work on non-intel CPUs) Closes #2029.
* | Merge branch 'nightly-ice' into 'main'gabi-2502025-06-092-3/+3
|\ \ | | | | | | | | | | | | | | | | | | ci: Unpin nightly version Closes #2018 See merge request tpo/core/arti!3035
| * | tor-circmgr: fix docs failureSteven Engler2025-06-091-2/+2
| | |
| * | tor-llcrypto: fix feature name typoSteven Engler2025-06-091-1/+1
| | |
* | | Merge branch 'flow-ctrl' into 'main'David Goulet2025-06-096-3/+191
|\ \ \ | |_|/ |/| | | | | | | | tor-cell: Add XON/XOFF cells See merge request tpo/core/arti!3031
| * | tor-proto: handle XON/XOFF messages in conflux seqno logicSteven Engler2025-06-091-1/+2
| | |
| * | tor-cell: add `Xon`/`Xoff` cellsSteven Engler2025-06-093-1/+185
| | |
| * | tor-cell: add experimental `flowctl-cc` featureSteven Engler2025-06-092-1/+4
| |/
* | Merge branch 'rtcompat-rustls' into 'main'wesleyac2025-06-092-3/+1
|\ \ | | | | | | | | | | | | tor-rtcompat: Remove rustls dependency in favor of futures-rustls See merge request tpo/core/arti!2943
| * | tor-rtcompat: Remove rustls dependency in favor of futures-rustlsparazyd2025-06-062-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | futures-rustls provides and re-exports rustls and thus the rustls dependency is redundant. Additionally, this removes `aws-lc-rs` and its inherent dependencies from `Cargo.lock` and the entire project since they've been included because the rustls dependency did not use `default-features = false` and rustls by default now depends on `aws-lc-rs` as its crypto engine.
* | | Merge branch 'less-noisy-pick-n-relays' into 'main'David Goulet2025-06-091-3/+4
|\ \ \ | |_|/ |/| | | | | | | | NetDir::pick_n_relays: avoid some unwarranted warnings See merge request tpo/core/arti!3028
| * | NetDir::pick_n_relays: avoid some unwarranted warningsJim Newsome2025-06-051-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously we logged a warning whenever `choose_multiple_weighted` returns `Ok(iter)`, whenever `iter` contains fewer relays than requested. However, this is expected behavior of `choose_multiple_weighted` if the filtered list of relays doesn't have the requested number of relays, in which case it should return all of the relays in the filtered list. In particular, this condition occurs regularly in test networks when looking for candidate relays to add to the potential guard set while excluding those already in the potential guard set. We still warn if the returned list is fewer than requested, *and* fewer than available in the filtered list.
* | | tor-proto: fix bug in `RateLimitedWriter::poll_write`Steven Engler2025-06-082-16/+100
| | | | | | | | | | | | | | | | | | Previously it did not behave correctly when `bucket.max()` was 0 (it would sleep for 0 time instead of infinitely, triggering a debug assertion).
* | | tor-proto: change `wake_when_bytes_available` to `NonZero<u64>`Steven Engler2025-06-082-6/+8
| | |
* | | tor-proto: add additional unit test for `TokenBucket::adjust`Steven Engler2025-06-081-1/+57
| | |
* | | tor-proto: improve `TokenBucket::adjust`Steven Engler2025-06-081-42/+87
| | | | | | | | | | | | The token bucket is now refilled before changing the rate.
* | | tor-proto: rename `TokenBucket::last_refill` to `added_tokens_at`Steven Engler2025-06-081-18/+18
| |/ |/|
* | Merge branch '1686a/expose-socks-proxy' into 'main'Nick Mathewson2025-06-051-3/+21
|\ \ | |/ |/| | | | | expose socks proxy some more See merge request tpo/core/arti!3016
| * expose socks proxy some moretrinity-1686a2025-05-301-3/+21
| |
* | tor-proto: return `Result` from `tokens_available_at`Steven Engler2025-06-052-32/+66
| |
* | tor-proto: change `RateLimitedWriter` logic to use user-configurable limitSteven Engler2025-06-052-16/+50
| | | | | | | | The user now sets a constant amount of bytes to wait for.
* | tor-proto: change `ClaimedTokens::commit_fewer` to `reduce`Steven Engler2025-06-052-21/+14
| |
* | tor-proto: add `{TokenBucket,RateLimitedWriter}Config` typesSteven Engler2025-06-053-33/+110
| |
* | tor-proto: add token bucket test for token/duration calcSteven Engler2025-06-051-0/+64
| |
* | tor-proto: add missing doc comment to `DataWriter::writer`Steven Engler2025-06-051-0/+1
| |
* | tor-proto: pass the time provider to the `DataWriter`Steven Engler2025-06-053-11/+28
| |
* | tor-proto: update doc comments for `DataWriter{,Inner}`Steven Engler2025-06-051-21/+25
| | | | | | | | | | Unfortunately the git diff thinks I moved the struct, but I really only moved the comment.