aboutsummaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | tor-netdoc: authcert tests: insist on return value typeIan Jackson2026-06-101-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're about to make UnverifiedAuthCert::verify return a TimerangeBound, not an AuthCert. For the success cases, declare the expected return type.
* | | | | | tor-dirserver: Promote tor-checkable depIan Jackson2026-06-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | We're going to need to talk about Timebound in the non-test code.
* | | | | | tor-checkable: TimerangeBound: is_valid: Treat bounds as inclusive (fmt)Ian Jackson2026-06-101-2/+6
| | | | | |
* | | | | | tor-checkable: TimerangeBound: is_valid: Treat bounds as inclusiveIan Jackson2026-06-101-2/+2
| |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes more sense because: * `start_bound` and `end_bound` return `Bound::Included` * It doesn't make sense to say that a document isn't valid at the very start of its validity period. (And we don't want a half-open range.) * NotYetValid(0ns) and Expired(0ns) are rather nonsensical errors. In real timeliness tests it will make no difference since the current time is only equal to the bound for a nanosecond. But tests that like to test corner cases might be affected, and possibly some synthetic situations (where the document's own validity time start is used somehow) might change.
* | | | | Merge branch 'clippy-string-slice' into 'main'Nick Mathewson2026-06-10384-0/+453
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Lint for clippy::string_slice See merge request tpo/core/arti!4086
| * | | | | everywhere: Add #[allow(clippy::string_slice)]Clara Engler2026-06-0926-0/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds #[allow(clippy::string_slice)] to all functions in the code where string slices are used, alongside a TODO comment. We do this add the function header to have it consistent, as things like expression based allow's are still experimental.
| * | | | | maint: Run maint/add_warning to deny string slicesClara Engler2026-06-09378-0/+411
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit executes maint/add_warning with the just added change to deny string slices except in tests. I recommend auditing this by checking out the previous commit followed by running the script yourself and then verifying that the diff is identical to this commit. This commit makes cargo clippy fail. We will add exceptions in the next commit.
* | | | | Merge branch 'cc-default' into 'main'Nick Mathewson2026-06-102-1/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Add 'flowctl-cc' to 'default' See merge request tpo/core/arti!4042
| * | | | | arti: add 'flowctl-cc' to 'default'Steven Engler2026-06-011-0/+1
| | | | | |
| * | | | | arti-client: add 'flowctl-cc' to 'default'Steven Engler2026-06-011-1/+1
| | | | | |
* | | | | | tor-netdoc: Fix a doc comment formatting botchIan Jackson2026-06-091-1/+1
| | | | | |
* | | | | | tor-netdoc: Introduce toplevel NeteworkStatus typeIan Jackson2026-06-095-0/+74
| |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All of the pieces are in place for this now. Some of the pieces are not entirely complete or correct at this point: that's sorted out by other MRs already open. Currently there is no encoding, no verification functdion, and no tests. Encoding depends on another oustanding MR (arti!4058). Verification depends on the rework in arti!4065. Tests are still to come.
* | | | | Merge branch 'fuzz-parse2' into 'main'Nick Mathewson2026-06-092-2/+10
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | netdoc: Add fuzzing support for parse2 with authcert, mds See merge request tpo/core/arti!4068
| * | | | | netdoc: Add fuzzing support for parse2 with authcert, mdsNick Mathewson2026-06-042-2/+10
| | | | | |
* | | | | | Merge branch 'deftly' into 'main'Nick Mathewson2026-06-0932-33/+33
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bump derive-deftly to 1.11.3 to get $impl See merge request tpo/core/arti!4083
| * | | | | | Bump derive-deftly to 1.11.3Ian Jackson2026-06-0932-33/+33
| | |_|/ / / | |/| | | | | | | | | | | | | | | | New beta semver policy means we should pin the patchlevel.
* | | | | | tor-netdoc: F64Finite, initially for votes' stats itemsIan Jackson2026-06-093-0/+135
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The actual item field isn't in the struct yet, because that would produce conflicts with other outstanding work in this same struct.
* | | | | | tor-netdoc: document new encoding implsIan Jackson2026-06-091-0/+2
| | | | | |
* | | | | | tor-netdoc: impl EncodeOrd for SignatureIan Jackson2026-06-091-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | This will let us encode consensus signatures sections.
* | | | | | tor-netdoc: impl EncodeOrd for RouterStatusIan Jackson2026-06-092-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | This defines ordering of routerstatus items.
* | | | | | tor-netdoc: derive encoding for RouterStatusIan Jackson2026-06-092-1/+1
| | | | | |
* | | | | | tor-netdoc: derive encoding for RouterStatusIntroItemIan Jackson2026-06-091-0/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | This is a nontrivial type with much weirdness to do with publication times etc., so let's mark this as incomplete until we have a round trip test.
* | | | | tor-netdoc: Reject Object again in ItemPresentClara Engler2026-06-091-3/+4
| | | | | | | | | | | | | | | | | | | | We must reject unknown objects unless noted otherwise.
* | | | | tor-netdoc: Allow args and object in ItemPresent (fmt)Clara Engler2026-06-091-8/+2
| | | | | | | | | | | | | | | | | | | | No functional change.
* | | | | tor-netdoc: Allow args and object in ItemPresentClara Engler2026-06-091-8/+7
| | | | | | | | | | | | | | | | | | | | | | | | | Changes ItemPresent to accept but ignore additional arguments and/or an object.
* | | | | tor-netdoc: Small ItemPresent test fixClara Engler2026-06-091-1/+1
| | | | | | | | | | | | | | | | | | | | Adds the actual keyword for a present object to the test.
* | | | | tor-netdoc: Test for ItemPresentClara Engler2026-06-091-1/+89
| | | | | | | | | | | | | | | | | | | | | | | | | This commit adds a test for ItemPresent in terms of encoding and decoding.
* | | | | tor-netdoc: Add ItemPresent typeClara Engler2026-06-093-2/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds the ItemPresent type, which is intended for token types where the mere presence implies a truthful value, such as `tunnelled-dir-server`. For now, it lives in `ignored_impls` because it does in fact ignore (actually rejects) everything around the item except the keyword. The type implements ItemValueParseable and ItemValueEncodable; both implementations being very minimal and only parsing/emitting the keyword. A test will be added within the next commit.
* | | | | tor-netdoc: Document Transparent shortcomingClara Engler2026-06-091-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit documents a shortcoming explained on IRC that exists with the current derive(Transparent) and generic types that are uncovered. See E0210.
* | | | | Merge branch 'netdoc-small-fixes' into 'main'Clara Engler2026-06-093-4/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tor-netdoc: minor fixes arising from netstatus work See merge request tpo/core/arti!4055
| * | | | | tor-netdoc: {required,recommended}-*-protocols: change orderingIan Jackson2026-06-031-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | This matches C Tor and the spec.
| * | | | | tor-netdoc: {required,recommended}-*-protocols: fix encodingIan Jackson2026-06-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | We need to use `-`, not `_`. So use concat! stringify!.
| * | | | | tor-netdoc: SoftwareVersion: fix Display implIan Jackson2026-06-031-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | Now it round-trips.
| * | | | | tor-netdoc: fix a doc formatting typoIan Jackson2026-06-031-1/+1
| | |_|_|/ | |/| | |
* | | | | Merge branch 'parse-input-improve' into 'main'Clara Engler2026-06-092-0/+17
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tor-netdoc: ParseInput: improve See merge request tpo/core/arti!4057
| * | | | | tor-netdoc: ParseInput: document changesIan Jackson2026-06-031-0/+1
| | | | | |
| * | | | | tor-netdoc: ParseInput: add retain_unknown_values convenience setterIan Jackson2026-06-031-0/+10
| | | | | |
| * | | | | tor-netdoc: ParseInput: add accessorsIan Jackson2026-06-031-1/+4
| | | | | |
| * | | | | tor-netdoc: ParseInput: add some blank linesIan Jackson2026-06-031-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes the code clearer, especially since we're going to add attributes.
| * | | | | tor-netdoc: ParseInput: impl Debug and CloneIan Jackson2026-06-031-0/+1
| |/ / / /
* | | | | proto: Move RateLimitedWriter to tor-async-utilsDavid Goulet2026-06-0811-40/+55
| | | | | | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
* | | | | proto: Move TokenBucket to tor-basic-utilsDavid Goulet2026-06-085-35/+31
| | | | | | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
* | | | | tor-rtcompat: change `Default` impls for connect/listen optionsSteven Engler2026-06-081-37/+60
| | | | | | | | | | | | | | | | | | | | The new `Default`s have no panicking code paths.
* | | | | tor-rtcompat: replace `_options` with `()`Steven Engler2026-06-081-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | This hopefully better shows that there's nothing to use here for general socket types.
* | | | | tor-rtcompat: updated semver.md for `connect()` changesSteven Engler2026-06-081-0/+3
| | | | |
* | | | | tor-rtcompat: add connect() options to set the socket buf sizesSteven Engler2026-06-082-2/+25
| | | | |
* | | | | tor-rtcompat: manually create and connect() socketSteven Engler2026-06-085-15/+156
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will allow us in the future to set custom sockopts on the socket before calling connect(). I tested an arti proxy with tokio and async-std manually. Arti doesn't yet support smol so I was not able to test it, but it's using the same code as async-std so I would expect it to work.
* | | | | tor-rtcompat+misc: add `NetStreamProvider::ConnectOptions`Steven Engler2026-06-0820-58/+203
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the trait type `ConnectOptions` to `NetStreamProvider` and adds this `ConnectOptions` as an argument to `NetStreamProvider::connect()`. You probably want to look at the changes in tor-rtcompat first, then the rest of this commit is updating the various places we use `NetStreamProvider`.
* | | | | tor-rtcompat: add structs to hold connect() optionsSteven Engler2026-06-082-1/+87
| | | | | | | | | | | | | | | | | | | | | | | | | For now this just sets up the structure. We'll add options for TCP later.
* | | | | tor-rtcompat: improve comment on `UnixListenOptions`Steven Engler2026-06-081-1/+1
| | | | |