aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | proto: Make the forward -> backward reactor chan boundedGabriela Moldovan2025-11-243-5/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This MPSC channel now has zero buffering, because if the backward reactor can't send the cell, the forward reactor must immediately stop reading from its inputs (note that the backward reactor's Tor channel towards the client *does* have a buffering layer).
| * | | | | proto: Let cell_rx carry more than just stream dataGabriela Moldovan2025-11-242-21/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are other types of messages (like circuit-level SENDMEs) that need to be handled in the backward reactor too.
| * | | | | proto: Give RelayReactor and BackwardReactor a RuntimeGabriela Moldovan2025-11-242-3/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We will soon need a handle to the runtime in BackwardReactor (we need a time provider to call `note_sendme_received()`).
| * | | | | proto: Don't poll PollAll unless we can write to chanGabriela Moldovan2025-11-241-14/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We need to avoid reading from all of these if the chan_sender isn't ready, because otherwise we can't provide backpressure.
| * | | | | proto: Use CircuitCellSender in the backward reactorGabriela Moldovan2025-11-241-8/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The backward reactor needs this for handling padding (because it may need to e.g. put the chan sender in a blocked state). Note that `CircuitCellSender` has an underlying `SometimesUnboundedSink`, so from now on we must be very careful and avoid writing to the `chan_sender` if it's not ready (because otherwise we can end up buffering unboundedly). The next commit will add some extra checks against reading from futures::Streams that might cause us to write to the `chan_sender` sink (futures::Sink).
| * | | | | proto: Return early if the cell is unrecognizedGabriela Moldovan2025-11-241-11/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rewriting the code this way reduces the indentation level for the "recognized cell" case (which is not yet implemented, but will be very soon), thereby making it more readable.
* | | | | | Merge branch 'p2-item-rest-with' into 'main'Ian Jackson2025-12-023-10/+16
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-netdoc: Rework ItemValueParseable, netdoc(rest, with) See merge request tpo/core/arti!3502
| * | | | | | parse2: ItemValueParseable, netdoc(object, rest): docs: state signatureIan Jackson2025-12-021-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3502/diffs?commit_id=f9a6d90ec6d5abf2958534d2c100c84f325e3634#2525aaf11b44162850101813478c4734160e11fa_920_919
| * | | | | | parse2: ItemValueParseable, netdoc(object, rest): docs: clarify errorsIan Jackson2025-12-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The error is not really discarded (which would seem to mean ignoring it). The erorr *value* is discarded and is replaced with a less informative error value.
| * | | | | | parse2: ItemValueParseable, netdoc(object, rest): docs: fix typoIan Jackson2025-12-021-1/+1
| | | | | | |
| * | | | | | parse2: ItemValueParseable, netdoc(rest, with): reworkIan Jackson2025-12-023-7/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The path in `with` must always be a module, not a function, because we need to use it for encoding too. In the case of `rest`, it's probably sensible to require a different function name, given that the semantics are subtly different to those of `rest` without `with`, since the latter can only get a single word but with `rest` you get the whole line.
| * | | | | | parse2: ItemValueParseable, netdoc(rest): docs: clarify behaviourIan Jackson2025-12-021-1/+1
| | | | | | |
* | | | | | | Merge branch 'static-arti-rpc-client-lib' into 'main'Nick Mathewson2025-12-021-1/+1
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | Added `staticlib` crate-type to arti-rpc-client-core which is needed for iOS. See merge request tpo/core/arti!3523
| * | | | | | Added `staticlib` crate-type to arti-rpc-client-core which is needed for iOS.Benjamin Erhart2025-12-021-1/+1
|/ / / / / /
* | | | | | Merge branch 'rm-semver' into 'main'Alexander Hansen Færøy2025-12-027-22/+0
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove semver.md files post-release See merge request tpo/core/arti!3520
| * | | | | | Remove semver.md files post-releaseGabriela Moldovan2025-12-027-22/+0
| | | | | | |
* | | | | | | Merge branch 'cargo-updates' into 'main'Alexander Hansen Færøy2025-12-022-208/+206
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | release: Bump metrics-exporter-prometheus See merge request tpo/core/arti!3521
| * | | | | | | Revert "release: Bump criterion to 0.8.0"Gabriela Moldovan2025-12-022-58/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See #2281 This reverts commit c00d381068cd8b88d6b12d8c7be8d4e1dafc828d.
| * | | | | | | release: Bump criterion to 0.8.0Gabriela Moldovan2025-12-022-4/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They've bumped their MSRV to 1.86 in this release, but that is okay, because we are also on 1.86. See the [criterion changelog] for more details. [criterion changelog]: https://github.com/criterion-rs/criterion.rs/blob/master/CHANGELOG.md#080---2025-11-29
| * | | | | | | release: Bump metrics-exporter-prometheus to 0.18.0Gabriela Moldovan2025-12-022-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Part of #2278
| * | | | | | | release: Run cargo updateGabriela Moldovan2025-12-021-205/+203
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Part of #2278
* | | | | | | Merge branch 'listen' into 'main'opara2025-12-024-114/+315
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-config: Refactor `Listen` to make usable for arti-relay See merge request tpo/core/arti!3469
| * | | | | | | tor-config: improve comments about adding functionalitySteven Engler2025-12-021-5/+7
| | | | | | | |
| * | | | | | | tor-config: add more comments to `ListenItem`Steven Engler2025-12-021-1/+10
| | | | | | | |
| * | | | | | | tor-config: rename `Listen::is_localhost_only` to `is_loopback_only`Steven Engler2025-12-024-4/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Assumes that 0.37.0 will be the next version number.
| * | | | | | | tor-config: add new `CustomizableListen`Steven Engler2025-12-021-101/+193
| | | | | | | |
| * | | | | | | tor-config: fix some localhost vs loopback names/commentsSteven Engler2025-11-141-4/+5
| | | | | | | |
| * | | | | | | tor-config: made `InvalidListen` privateSteven Engler2025-11-142-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was no reason for this to be public.
| * | | | | | | tor-config: simplify a match statementSteven Engler2025-11-141-8/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This does a little extra cloning, but I think is easier to read.
| * | | | | | | tor-config: add serde error messages to `ListenSerde` and `ListenItemSerde`Steven Engler2025-11-141-2/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For example the old message would look like: ```text target/debug/arti: error: read configuration: Config contents not as expected: Couldn't load configuration: data did not match any variant of untagged enum ListenSerde for key "default.proxy.socks_listen" in command line ``` The new message looks like: ```text target/debug/arti: error: read configuration: Config contents not as expected: Couldn't load configuration: value was not a bool, `u16` integer, string, or list of integers/strings for key "default.proxy.socks_listen" in command line ```
| * | | | | | | tor-config: add more tests for `Listen`Steven Engler2025-11-141-0/+81
| | | | | | | |
| * | | | | | | tor-config: remove unneeded `Serialize`/`Deserialize`Steven Engler2025-11-131-2/+1
| | | | | | | |
* | | | | | | | Merge branch 'shadow-ci-thresholds' into 'main'Jim Newsome2025-12-021-11/+11
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | shadow CI: tweak HS required success thresholds See merge request tpo/core/arti!3515
| * | | | | | | shadow CI: *increase* required success rate for c-tor clientJim Newsome2025-12-021-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | None of the failures currently recorded in https://gitlab.torproject.org/tpo/core/arti/-/issues/2209 involve the c-tor client. I think we can safely require all xfers to succeed for the c-tor client.
| * | | | | | | shadow CI: reduce arti client HS required successesJim Newsome2025-12-021-7/+7
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | From 9/10 to 5/10. This is to mitigate flakiness. See https://gitlab.torproject.org/tpo/core/arti/-/issues/2209#note_3295789
* | | | | | | Merge branch 'shadow-test-run-no-echo' into 'main'wesleyac2025-12-021-5/+7
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | ci: Disable echoing in shadow test script. See merge request tpo/core/arti!3496
| * | | | | | ci: Improve formatting of shadow results.Wesley Aptekar-Cassels2025-11-251-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Having this all on one line makes it easier to read.
| * | | | | | ci: Print host names in shadow run script.Wesley Aptekar-Cassels2025-11-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that we no longer echo these, this is useful.
| * | | | | | ci: Print seed in shadow run script.Wesley Aptekar-Cassels2025-11-251-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that we no longer echo this, it's potentially useful to have it.
| * | | | | | ci: Disable echoing in shadow test script.Wesley Aptekar-Cassels2025-11-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I find that the echoing here does not really contribute to my understanding of what's going on, and it makes the output quite difficult to parse.
* | | | | | | Merge branch 'release-date' into 'main'arti-v1.8.0Alexander Hansen Færøy2025-12-021-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Run maint/update-release-date See merge request tpo/core/arti!3519
| * | | | | | | Run maint/update-release-date.Gabriela Moldovan2025-12-021-1/+1
| | | | | | | |
* | | | | | | | Merge branch 'version-bumps-arti-1.8.0' into 'main'gabi-2502025-12-0274-679/+679
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Version bumps for the arti 1.8.0 release See merge request tpo/core/arti!3518
| * | | | | | | Bump arti crate to 1.8.0Gabriela Moldovan2025-12-024-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Done using: ``` cargo set-version --bump minor -p arti ```
| * | | | | | | Bump all the unstable tor- and arti- crates to 0.37.0.Gabriela Moldovan2025-12-0262-564/+564
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Done using: ``` for crate in $(./maint/list_crates | rg '^(tor|arti-)'); do cargo set-version -p $crate 0.37.0 done
| * | | | | | | Run cargo update for {hashx,equix}/benchGabriela Moldovan2025-12-022-73/+73
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Done using ``` (cd crates/hashx/bench && cargo update) (cd crates/equix/bench && cargo update) ```
| * | | | | | | Bump the versions of the non-{arti-,tor-} cratesGabriela Moldovan2025-12-0226-38/+38
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The non-{arti-,tor-} crates are: ``` ./maint/list_crates | rg -v '^(tor|arti)' oneshot-fused-workaround slotmap-careful test-temp-dir fslock-guard hashx equix caret fs-mistrust safelog retry-error futures-copy ``` We split them in the following categories: * crates with no changes (no version bumps): None ``` maint/changed_crates -v "arti-v$LAST_VERSION" 2>&1 >/dev/null | grep -i "no change" | grep -v '\(tor\|arti\)-' ``` * crates that only have non-functional changes (bump the patch version, but not the dependend-on version): - oneshot-fused-workaround - slotmap-careful - test-temp-dir - fslock-guard - equix - caret - safelog - retry-error * crates where functional changes were made, but no APIs were added or broken: - hashx - fs-mistrust - futures-copy * crates where APIs were broken (bump minor): None The bumps from this commit were created using this script: ``` PATCH_NF=( oneshot-fused-workaround slotmap-careful test-temp-dir fslock-guard equix caret safelog retry-error ) PATCH=" hashx fs-mistrust futures-copy " ./maint/bump_nodep "${PATCH_NF[@]}" for crate in $PATCH; do cargo set-version --bump patch -p $crate; done ```
* | | | | | | Merge branch 'changelog-1.8.0' into 'main'gabi-2502025-12-021-1/+314
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add changelog for 1.8.0 See merge request tpo/core/arti!3514
| * | | | | | | changelog: Apply maint/format_md_linksGabriela Moldovan2025-12-011-3/+3
| | | | | | | |
| * | | | | | | changelog: Add spacing in proposal nameGabriela Moldovan2025-12-011-2/+2
| | | | | | | |