summaryrefslogtreecommitdiff
path: root/crates
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.
* | | | | 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-082-8/+61
| | | | |
* | | | | 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 '2151-check-integrity-output' into 'main'opara2025-10-081-55/+183
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| | | | | |
| * | | | | 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
| | | | | |