summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | tor-netdoc: parse2 tests: Have t_err_* return the error (pre-fmt)Ian Jackson2025-10-091-1/+6
| | | | |
| * | | | tor-netdoc: parse2 tests: Check columnsIan Jackson2025-10-091-7/+15
| | | | |
| * | | | tor-netdoc: parse2 tests: Check columns (pre-fmt)Ian Jackson2025-10-091-1/+2
| | | | |
| * | | | tor-netdoc: parse2: Add column field to PaarsError and print itIan Jackson2025-10-093-2/+14
| | | | |
| * | | | tor-netdoc: parse2: Add .column() accessor to ErrorProblemIan Jackson2025-10-092-1/+29
| | | | | | | | | | | | | | | | | | | | We'll want to general way to find the error problem column.
| * | | | tor-netdoc: parse2: Add column field as far as ErrorProblemIan Jackson2025-10-093-7/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Provide facilities in ArgumnetStream for determining the column, and add the field to all the error types up to and including ErrorProblem. The value still needs to be propagated to ParseError, and printed there.
| * | | | tor-netdoc: parse2: Replace EP with AE in derive fields rest pathIan Jackson2025-10-091-1/+2
| | | | |
| * | | | tor-netdoc: parse2: Replace EP with AE in NdiDirectorySignature (fmt)Ian Jackson2025-10-091-7/+10
| | | | |
| * | | | tor-netdoc: parse2: Replace EP with AE in NdiDirectorySignatureIan Jackson2025-10-091-4/+4
| | | | |
| * | | | tor-netdoc: Abolish now-obsolete field argument to from_args (fmt)Ian Jackson2025-10-094-29/+7
| | | | |
| * | | | 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
| | | | | | |