aboutsummaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Merge branch 'testdata-export-bis' into 'main'Ian Jackson2026-07-2732-484/+728
|\ \ \ | | | | | | | | | | | | | | | | tor-netdoc testdata-live: More shell script, less macrology See merge request tpo/core/arti!4238
| * | | tor-netdoc testdata-live: Abolish selected_relays.rsIan Jackson2026-07-272-27/+0
| | | | | | | | | | | | | | | | | | | | This is no longer used. Get rid of the code which generates it, and the generated file.
| * | | tor-netdoc testdata-live: Switch to generated_consts.rsIan Jackson2026-07-271-61/+7
| | | | | | | | | | | | | | | | | | | | This gits rid of the macrology, apart from the generated macro module file.
| * | | tor-netdoc testdata-live-download: Generate generated_consts.rsIan Jackson2026-07-272-1/+324
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the code for the new approach. It's kind of ad-hoc but also much less deep than the previous macrology. And, run the script to see the output. Currently that file isn't used. For now we leave the shell script code for generating selected_relays.rs in testdata-live-download. The new code for generated_consts.rs is very different, so it's clearer to split the changes up this way.
| * | | tor-netdoc testdata-live: Run a downloadIan Jackson2026-07-2728-396/+398
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I'm going to change the generation, and that means I need input files, so I need to run a download (as per the comment in testdata-live-download). I'm doing this first, so the rest of the MR can see individual changes.
* | | | Merge branch 'testdata-report-nits' into 'main'Ian Jackson2026-07-271-1/+5
|\| | | | |/ / |/| | | | | | | | tor-netdoc testdata-live: Two nits See merge request tpo/core/arti!4237
| * | tor-netdoc testdata-live: Add a comment about running without downloadIan Jackson2026-07-271-0/+4
| | |
| * | tor-netdoc testdata-live: Fail if curl failsIan Jackson2026-07-271-1/+1
| | |
* | | Merge branch 'remove-expect-unused' into 'main'Ian Jackson2026-07-272-2/+2
|\ \ \ | | | | | | | | | | | | | | | | arti-relay: Remove unused_async clippy escape hatch See merge request tpo/core/arti!4239
| * | | arti-relay: Add back allow(clippy::unused_async)Gabriela Moldovan2026-07-272-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | The clippy CI job that uses RECENT_RUST_IMAGE fails without the `allow` (RECENT_RUST_IMAGE uses Rust 1.96, where the `unused_async` false-positive still appears to trigger).
| * | | arti-relay: Remove unused_async clippy escape hatchGabriela Moldovan2026-07-272-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was needed to work around two `unused_async` false-positives that would trigger on older Rust versions. This was fixed in https://github.com/rust-lang/rust-clippy/issues/15305 and released in 1.90, according to the [changelog]. Since these no longer trigger on 1.97.1, they are now causing the `expect(clippy::unused_async)` to fail. Removing the `expect`s seems like the most sensible fix. [changelog]: https://github.com/rust-lang/rust-clippy/issues/15305
* | | | Merge branch 'netdoc-affordances' into 'main'Ian Jackson2026-07-276-20/+38
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | tor-netdoc: Add a few affordances See merge request tpo/core/arti!4235
| * | | | tor-netdoc: docs: fix typos and linkifyIan Jackson2026-07-271-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4235#note_3439437 https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4235#note_3439438
| * | | | tor-netdoc: Don't writhe in a doc commentIan Jackson2026-07-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4235#note_3439436
| * | | | tor-netdoc: derive Hash for netstatus::ConsensusMethodsIan Jackson2026-07-232-1/+2
| | | | |
| * | | | tor-netdoc: Use encode_netdoc_unsigned in several placesIan Jackson2026-07-233-19/+9
| | | | |
| * | | | tor-netdoc: Provide free functions for encodingIan Jackson2026-07-232-0/+27
| |/ / / | | | | | | | | | | | | | | | | I keep finding I want to encode things and then I have to prat about with a NetdocEncoder. Let's provide potted versions.
* | | | Merge branch 'testdata-export' into 'main'Ian Jackson2026-07-2765-391/+508
|\ \ \ \ | | |/ / | |/| | | | | | | | | | tor-netdoc testdata-live: Export for the benefit of other crates See merge request tpo/core/arti!4229
| * | | tor-netdoc testdata-live: Export for the benefit of other cratesIan Jackson2026-07-232-0/+89
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a new testdata_live module which is exposed with the testing features, containing the testdata-live in string constants. This avoids the need for test cases in other crates to walk the filesystem to an area outside their own crate path.
| * | | Require derive-deftly 1.11.4Ian Jackson2026-07-2334-35/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're about to *use* the fix that was in 1.11.4. We already updated the lockfile in !4228, so this is just Cargo.toml changes (which will prevent the minimal-versions tests failing when we change the code to rely on the fix). git-grep -l '^derive-deftly' |xargs perl -i~ -pe 's{\b\Q1.11.3\E\b}{1.11.4}g'
| * | | tor-netdoc testdata-live: Run a downloadIan Jackson2026-07-2328-356/+367
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This updates everything, but really I'm just running it to create the new selected_relays.rs file. The meaning of this file will become clear shortly.
| * | | tor-netdoc testdata-live: Write out a selected_relays.rs fileIan Jackson2026-07-231-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We could use build.rs for this, but since we have a script already, this seems better. The meaning of this file will become clear shortly.
* | | | Merge branch 'promote-supported-consensus' into 'main'gabi-2502026-07-276-17/+37
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Move supported consensus methods iteration to tor-dirauth from arti-dirauth See merge request tpo/core/arti!4234
| * | | | Move supported consensus methods iteration to tor-dirauth from arti-dirauthIan Jackson2026-07-236-17/+37
| | |/ / | |/| | | | | | | | | | Now we provide SupportedConsensusMethod::iter_all().
* | | | arti: Ratelimit proxy error reportssyphyr2026-07-232-3/+47
| | | | | | | | | | | | | | | | Implement log_ratelim! in report_proxy_error
* | | | proto: Include the actual cell command in an error messageGabriela Moldovan2026-07-231-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This can happen if we get an unexpected BEGIN_DIR/RESOLVE too, so we can't hard-code "BEGIN" in the error message. Context: https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4230#note_3439258,
* | | | proto: Remove feature-gating from CircHop::handle_msg()Gabriela Moldovan2026-07-233-6/+4
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This feature-gating has been a source of confusion, and it unnecessarily complicates the stream message handling flow. I've previously argued in favour of keeping it, in the spirit of a belt and braces approach to message validation, but I've been convinced that in this particular case, the feature-gate is more trouble than it's worth. What makes things worse is that the `CircHop::handle_msg()` function was designed poorly (by yours truly). I plan on refactoring it at some point, hopefully soon. There is a TODO about this below its doc comment.
* | | Merge branch 'arti-authority-plugin-1' into 'main'Ian Jackson2026-07-236-0/+482
|\ \ \ | | | | | | | | | | | | | | | | arti-dirauth: Build an arti consensus method plugin binary, and implement list-methods See merge request tpo/core/arti!4225
| * | | arti-dirauth: Correct URL for authority-plugin.mdIan Jackson2026-07-231-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This will appear when the MR is merged. As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4225#note_3439197
| * | | arti-dirauth: FilenameOrStdio: Add doc and comments about permissionsIan Jackson2026-07-231-0/+3
| | | |
| * | | arti-dirauth: FilenameOrStdio: Discussion of leftover .tmp filesIan Jackson2026-07-231-0/+11
| | | |
| * | | arti-dirauth: FilenameOrStdio: Clarify a TODOIan Jackson2026-07-231-1/+2
| | | |
| * | | arti-dirauth: Provide a command to dump the specIan Jackson2026-07-231-0/+21
| | | | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4225#note_3438787
| * | | arti-dirauth: Move plugin spec to crate subdirIan Jackson2026-07-231-0/+250
| | | |
| * | | arti-dirauth: Add a divider commentIan Jackson2026-07-231-0/+2
| | | | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4225#note_3438786
| * | | arti authority plugin: Implement list-methodsIan Jackson2026-07-212-4/+27
| | | |
| * | | arti authority plugin: FilenameOrStdio utilityIan Jackson2026-07-212-0/+66
| | | | | | | | | | | | | | | | | | | | | | | | Although we have other places in-tree where we do this very standard thing, we don't seem to have an affordance for it. I doubt we want to add a dependency just for this, so roll our own.
| * | | arti authority plugin: Command line parsing skeletonIan Jackson2026-07-212-1/+78
| | | |
| * | | arti-dirauth: Build an arti consensus method plugin binaryIan Jackson2026-07-214-0/+28
| | | | | | | | | | | | | | | | This just panics, right now.
* | | | Merge branch 'refactor-tor-dirmgr-0' into 'main'Jim Newsome2026-07-231-102/+180
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | tor-dirmgr: Refactor function download See merge request tpo/core/arti!4204
| * | | | tor-dirmgr: Refactor bootstrap::downloadhjrgrn2026-07-231-102/+180
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | - Add helper functions: perform_download, advance_state, apply_state, and update_state - Add enum used in said helper functions: DownloadOutcome and AdvanceStateError - Add minor improvements on the readability of the entire sub-module
* | | | Merge branch 'dynasm' into 'main'opara2026-07-233-25/+25
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | hashx: Update dynasmrt to 5.1.0 Closes #2637 See merge request tpo/core/arti!4232
| * | | | hashx,equix: update bench Cargo.lockSteven Engler2026-07-232-24/+24
| | | | | | | | | | | | | | | | | | | | This was forgotten in the previous commit.
| * | | | hashx: update dynasmrt to 5.1.0Steven Engler2026-07-231-1/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | This fixes a rust future-incompatibilities warning, which was caused by the proc-macro-error2 crate. The dynasmrt crate switched to a fork proc-macro-error3 to fix this.
* | | | Merge branch 'rsa-1024' into 'main'David Goulet2026-07-231-1/+21
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-cert-x509: Fix RSA key size check Closes #2626 See merge request tpo/core/arti!4231
| * | | | tor-cert-x509: fix size check in `create_legacy_rsa_id_cert`Steven Engler2026-07-231-3/+5
| | | | |
| * | | | tor-cert-x509: add unit test demonstrating bugSteven Engler2026-07-231-0/+18
| |/ / /
* | | | tor-netdoc: HsDesc parse_decrypt_validate: Fix docs re time boundsIan Jackson2026-07-231-1/+2
| | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4223#note_3438348
* | | | tor-checkable: Fix a wrong ref in semver.mdIan Jackson2026-07-231-1/+1
| | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4223#note_3438344
* | | | tor-checkable: Improve/fix a commentIan Jackson2026-07-231-1/+2
| | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4223#note_3438341