aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | tor-netdoc: Abolish now-obsolete field argument to from_argsIan Jackson2025-10-098-20/+12
| | | | | |
| * | | | | tor-netdoc: Introduce ArgumentError (fmt)Ian Jackson2025-10-091-3/+1
| | | | | |
| * | | | | tor-netdoc: Introduce ArgumentErrorIan Jackson2025-10-0910-33/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will allow us to (1) move field name handling out of the argument parser method, and also (2) add columns more easily.
| * | | | | tor-netdoc: Have .reject_extra_args return a bespoke errorIan Jackson2025-10-094-4/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're about to have some functions return ArgumentError and some ErrorProblem. We don't want to provide a conversion directly between these types, so instead introduce a new type that will be convertible to either.
| * | | | | tor-netdoc: UnparsedItem: Provide .args() accessorIan Jackson2025-10-092-0/+8
| | | | | |
| * | | | | tor-netdoc: parse2: Record previous position in ArgumentStream (fmt)Ian Jackson2025-10-091-1/+5
| | | | | |
| * | | | | tor-netdoc: parse2: Record previous position in ArgumentStreamIan Jackson2025-10-091-1/+8
| | | | | |
| * | | | | tor-netdoc: parse2: Rename methods on ArgumentStreamIan Jackson2025-10-093-11/+16
| | | | | |
| * | | | | tor-netdoc: parse2: Add whole_line_len to ArgumentStreamIan Jackson2025-10-092-4/+17
|/ / / / /
* | | | | Merge branch 'comparison-chain' into 'main'gabi-2502025-10-091-0/+6
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tor-proto: allow comparison chain See merge request tpo/core/arti!3342
| * | | | | tor-proto: bring back comparison chain w/ allowhashcatHitman2025-10-081-9/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was decided that the comparison chain was actually preferrable for readability. So instead, we're just `allow`ing it until it stops being a problem. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | tor-proto: replace comparison chain with matchhashcatHitman2025-10-081-8/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was a comparison chain in `tor_proto::util::poll_all::test::ResolveAfter::poll` which was causing a clippy warning. The lint in question, `clippy::comparison_chain`, was a `clippy::style` lint in 1.85.1 and got moved to `clippy::pedantic` in 1.87.0 (see [rust-clippy!14219]). Since some of us (like me) develop on MSRV, I'm fixing this lint now. Gabi didn't have any strong opinions on whether I did it like this or with an `allow` attribute, so I decided this was better since it means we don't have to come back later just to remove the `allow`. It should be noted that using a match like this can sometimes be a performance regression (see [rust-clippy#5354] and [rust-clippy!6390]). I would expect in this case the effect will be very little, if any, but if tests in `tor_proto::util::poll_all::test` start taking much longer and having an impact on CI or something, this could be why. [rust-clippy!14219]: https://github.com/rust-lang/rust-clippy/pull/14219 [rust-clippy#5354]: https://github.com/rust-lang/rust-clippy/issues/5354 [rust-clippy!6390]: https://github.com/rust-lang/rust-clippy/pull/6390 Signed-off-by: hashcatHitman <[email protected]>
* | | | | | Merge branch 'normal-ip-arguments' into 'main'Ian Jackson2025-10-091-0/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-netdoc: impl NormalItemArgument for Ipv4Addr and SocketAddr See merge request tpo/core/arti!3343
| * | | | | | tor-netdoc: impl NormalItemArgument for Ipv4Addr and SocketAddrIan Jackson2025-10-081-0/+5
| |/ / / / / | | | | | | | | | | | | | | | | | | AFAICT we don't have variant parsings for these things.
* | | | | | Merge branch 'relay-bin-4' into 'main'opara2025-10-086-49/+195
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | arti-relay: Add mainloop and prepare to begin bootstrapping work See merge request tpo/core/arti!3313
| * | | | | | arti-relay: replaced `Runtime + ToplevelBlockOn` with `ToplevelRuntime`Steven Engler2025-10-081-3/+3
| | | | | | |
| * | | | | | arti-relay: expand comment on `InertTorRelay`Steven Engler2025-10-081-0/+17
| | | | | | |
| * | | | | | arti-relay: change `allow` to `expect`Steven Engler2025-10-081-5/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | And remove one that we didn't need.
| * | | | | | arti-relay: add a new `TorRelay`Steven Engler2025-10-082-1/+50
| | | | | | |
| * | | | | | arti-relay: store the `TorRelayConfig` in the `InertTorRelay`Steven Engler2025-10-082-4/+7
| | | | | | |
| * | | | | | arti-relay: change `TorRelay` into `InertTorRelay`Steven Engler2025-10-082-24/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We'll add a `TorRelay` back in a following commit.
| * | | | | | tor-config: remove stale comment on `ConfigurationSources::try_from_cmdline`Steven Engler2025-10-081-2/+0
| | | | | | |
| * | | | | | arti-relay: log the override options usedSteven Engler2025-10-081-1/+32
| | | | | | |
| * | | | | | tor-config: add `ConfigurationSources::options()`Steven Engler2025-10-081-0/+7
| | | | | | |
| * | | | | | arti-relay: add mainloopSteven Engler2025-10-083-8/+63
| | | | | | |
| * | | | | | arti-relay: add `ToplevelBlockOn` to returned runtimeSteven Engler2025-10-081-2/+2
| | | | | | |
| * | | | | | arti-relay: don't prepend "arti-relay: " to log linesSteven Engler2025-10-081-10/+2
| | | | | | |
| * | | | | | arti-relay: re-add trailing '/' to default paths in help textSteven Engler2025-10-081-2/+12
|/ / / / / /
* | | | | | Merge branch 'aarch64-fix' into 'main'Nick Mathewson2025-10-083-13/+31
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | hashx: Fix compilation on aarch64 with dynasm 4.0 Closes #2208 See merge request tpo/core/arti!3341
| * | | | | | hashx: A couple of comments about register rangesNick Mathewson2025-10-082-0/+4
| | | | | | |
| * | | | | | hashx: Make exceptions for clippy warnings from dynasm.Nick Mathewson2025-10-081-9/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new conversion mechanisms in dynasm 4.0 make clippy unhappy under aarch64.
| * | | | | | hashx: Make RegisterId::x() return u8 on aarch64Nick Mathewson2025-10-071-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (Starting with version 4, dynasm wants something that implements Into<u8>.)
| * | | | | | hashx: Provide a u8 accessor for RegisterId.Nick Mathewson2025-10-072-1/+7
| | | | | | |
* | | | | | | Merge branch 'fewer-cpus' into 'main'Jim Newsome2025-10-082-12/+193
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CI: restrict cargo parallelism based on available memory See merge request tpo/core/arti!3332
| * | | | | | | CI: increase timeout to 2h on some jobsJim Newsome2025-10-081-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is to mitigate https://gitlab.torproject.org/tpo/tpa/team/-/issues/42317 I added the 2h timeout (overriding the default 1h) for all jobs that were cited as examples on that issue, and a couple others that looked likely to be time-consuming.
| * | | | | | | CI: dump /proc/loadavg and /proc/pressure/* for debugging purposesJim Newsome2025-10-081-0/+6
| | | | | | | |
| * | | | | | | Revert restricting heavy cargo jobs to TPA runnersJim Newsome2025-10-081-12/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * This reverts commit 8da606966e5213cada270fb64f896f69bab041dd. * This reverts commit a3ac2080956b9295000bd7011c34111a6d5ce5a7.
| * | | | | | | CI: limit cargo parallelism based on memory and ncpuJim Newsome2025-10-082-0/+171
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See https://gitlab.torproject.org/tpo/tpa/team/-/issues/42283
* | | | | | | | Merge branch '2151-check-integrity-output' into 'main'opara2025-10-082-243/+274
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | arti: keys: Rework the output of `arti keys check-integrity` Closes #2151 See merge request tpo/core/arti!3286
| * | | | | | | arti: keys: Simplify `run_check_integrity`hjrgrn2025-10-081-9/+7
| | | | | | | |
| * | | | | | | doc: Remove extra new line in `keys.md`hjrgrn2025-10-081-1/+0
| | | | | | | |
| * | | | | | | arti: keys: Improve readability of `print_check_integrity_incipit`hjrgrn2025-10-081-11/+7
| | | | | | | |
| * | | | | | | arti: keys: Fix typo in `run_check_integrity`hjrgrn2025-10-081-1/+1
| | | | | | | |
| * | | | | | | arti: keys: Rename `invalid_entry` to `expired_entry` in `run_check_integrity`hjrgrn2025-10-081-3/+3
| | | | | | | |
| * | | | | | | arti: keys: Add warning for unexpected invalid keystore entryhjrgrn2025-10-081-14/+16
| | | | | | | |
| * | | | | | | arti: keys: Improve readability of `run_check_integrity`hjrgrn2025-10-081-5/+4
| | | | | | | |
| * | | | | | | arti: keys: Add comment explaining why `services` cannot be droppedhjrgrn2025-10-081-0/+2
| | | | | | | |
| * | | | | | | arti: keys: Add comment explaining difference between `affected_keystores` ↵hjrgrn2025-10-081-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | and `keystores`
| * | | | | | | doc: Update `check-integrity` section in `keys.md`hjrgrn2025-10-081-187/+91
| | | | | | | |
| * | | | | | | arti: keys: Refactor `get_expired_keys`hjrgrn2025-10-081-9/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The function now returns `Result<Vec<InvalidKeystoreEntry<'a>>>`