aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | tor-netdoc: tests: ns parse2 test: Mess with authcerts in a blockIan Jackson2026-06-171-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This avoids rebinding file and text and input to the authcerts. We're going to want to use at least some of those values later.
| * | | | | | | | tor-netdoc: tests: ns parse2 test: enable UnknownIan Jackson2026-06-171-1/+2
| | | | | | | | |
| * | | | | | | | tor-netdoc: tests: ns parse2 test: Use real document type (fmt)Ian Jackson2026-06-171-3/+2
| | | | | | | | |
| * | | | | | | | tor-netdoc: tests: ns parse2 test: Use real document typeIan Jackson2026-06-171-4/+4
| | | | | | | | |
| * | | | | | | | tor-netdoc: tests: Move ns poc tests to netstatus.rsIan Jackson2026-06-173-70/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're going to turn these into full-on round-trip tests for both flavours of consensus. We must temporarily bodge the more comprehensive test, for plain con sensuses, to to still use the actual document type from poc. That allows us to avoid changing the arguments to verify() etc. (poc's verify is subtly different to the real NetworkStatus's.) When these tests were in poc, they were gated by the incomplete cfg on the whole of poc. Now they have one each. Almost entirely code motion; review with git show --color-moved --color-moved-ws=allow-indentation-change
| * | | | | | | | tor-netdoc: tests: Move poc authcert test to authcert.rsIan Jackson2026-06-172-16/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no separate poc authcert type any more. This is a perfectly fine test case. Almost entirely code motion; review with git show --color-moved --color-moved-ws=allow-indentation-change
| * | | | | | | | tor-netdoc: poc tests: drop some unneeded &'sIan Jackson2026-06-171-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | clippy complains (clippy::needless_borrows_for_generic_args). In poc, we have an allow with a TODO asking for this to be done generally in the Arti codebase. Let's postpone shaving that yak.
| * | | | | | | | tor-netdoc: poc tests: use normal names for authcertsIan Jackson2026-06-171-6/+4
| | | | | | | | |
| * | | | | | | | tor-netdoc: poc tests: simulate normal outside-poc variety module namesIan Jackson2026-06-171-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We are going to reuse/move this test code. Ie, it is going to be testing the real network document types. The naming outside poc is different. To make future diffs a bit less noisy, change some type names in the test code and fudge it up with a use.
| * | | | | | | | tor-netdoc: un-cfg-gate some use'sIan Jackson2026-06-171-5/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have the #1645 allow at the top of lib.rs, so this is unnecessary bureaucracy. Let's simplify this before adding more to it.
| * | | | | | | | tor-netdoc: provide verify methods on new consensus types (fmt)Ian Jackson2026-06-171-3/+2
| | | | | | | | |
| * | | | | | | | tor-netdoc: provide verify methods on new consensus typesIan Jackson2026-06-172-2/+116
| | | | | | | | |
| * | | | | | | | tor-netdoc: Break out consensus_threshold fnIan Jackson2026-06-172-3/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This calculation appeared 3 times.
| * | | | | | | | tor-netdoc: NetworkStatus: derive EncodeIan Jackson2026-06-171-1/+1
| | | | | | | | |
* | | | | | | | | Merge branch 'sp-fingerprint-encode' into 'main'Clara Engler2026-06-172-5/+32
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Encode for SpFingerprint See merge request tpo/core/arti!4116
| * | | | | | | | | tor-netdoc: Round-trip test SpFingerprint encodingClara Engler2026-06-171-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds a round-trip encoding for SpFingerprint to the already existing parsing test.
| * | | | | | | | | tor-netdoc: Small SpFingerprint test refactoringClara Engler2026-06-171-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not shadow s and provide access to Wrapper instead of just the inner fingerprint. We will make use of this soon.
| * | | | | | | | | tor-netdoc: Encode for SpFingerprintClara Engler2026-06-172-0/+16
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements encode::ItemArgument for SpFignerprint. A test will be added in the next commit.
* | | | | | | | | Merge branch 'ntor-cc-rd' into 'main'Ian Jackson2026-06-173-7/+25
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | Add ntor onion key cross certificate to RouterDesc See merge request tpo/core/arti!4109
| * | | | | | | | tor-netdoc: Create NtorOnionKeyCrossCert at better placeClara Engler2026-06-171-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit moves the construction place for the NtorOnionKeyCrossCert from the final RouterDesc construction to the crosscert code block, which should be more correct.
| * | | | | | | | tor-netdoc: Add ntor_onion_key_crosscert to RouterDescClara Engler2026-06-152-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds ntor_onion_key_crosscert to RouterDesc as well as integrating it into the legacy parser.
| * | | | | | | | tor-netdoc: Extract ntor bit and cert in legacy parser (fmt)Clara Engler2026-06-151-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | No functional change.
| * | | | | | | | tor-netdoc: Extract ntor bit and cert in legacy parserClara Engler2026-06-151-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We will soon need it for the ntor cross certificate.
| * | | | | | | | tor-netdoc: Make dangerous_new_unverified available outside testsClara Engler2026-06-151-2/+4
| | |_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We will need it for compatibility with the legacy parser soon. Not super nice.
* | | | | | | | Merge branch 'trivial-encode-derive' into 'main'Clara Engler2026-06-172-3/+6
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Trivial ItemValueEncodable dervies See merge request tpo/core/arti!4115
| * | | | | | | | tor-netdoc: ItemValueEncodable for ExtraInfoDigestsClara Engler2026-06-172-1/+2
| | | | | | | | |
| * | | | | | | | tor-netdoc: ItemValueEncodable for RouterDescIntroItemClara Engler2026-06-172-1/+2
| | | | | | | | |
| * | | | | | | | tor-netdoc: ItemValueEncodable for OverloadGeneralClara Engler2026-06-172-1/+2
|/ / / / / / / /
* | | | | | | | Merge branch 'display-range' into 'main'Clara Engler2026-06-172-24/+61
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-netdoc: port policies: Encode with reject if it's shorter See merge request tpo/core/arti!4108
| * | | | | | | | tor-netdoc: placate clippyIan Jackson2026-06-161-1/+1
| | | | | | | | |
| * | | | | | | | tor-netdoc: port policies: Encode with reject if it's shorterIan Jackson2026-06-151-9/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When trying to repro a live consensus, I discovered that `reject 25` turned into `accept 1-24,26-65535`.
| * | | | | | | | tor-netdoc: port policies: tests: reformatIan Jackson2026-06-151-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use // to force rustdoc to multi-line layout. That makes the layout uniform across all these test cases, and will make the next commit clearer.
| * | | | | | | | tor-netdoc: port policies: Provide .invertedIan Jackson2026-06-151-3/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We'll want this in a moment, not just invert in place. I was tempted to remove the mutating form, but there are at least two call sites.
| * | | | | | | | tor-netdoc: port policies: Simplify display with iter_joinIan Jackson2026-06-151-6/+2
| | | | | | | | |
| * | | | | | | | tor-netdoc: port policies: Declare iter return value as CloneIan Jackson2026-06-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's Copy, in fact. But Copy iterators are a hazard.
| * | | | | | | | tor-netdoc: port policies: Provide display method on PortRangesIan Jackson2026-06-152-11/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're going to want this for a more clever formatting algorithm.
* | | | | | | | | Merge branch 'ahf/fix-wasm-build' into 'main'David Goulet2026-06-161-1/+1
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | Use wasm_js for getrandom in tor-bytes. See merge request tpo/core/arti!4119
| * | | | | | | | Use wasm_js for getrandom in tor-bytes.Alexander Hansen Færøy2026-06-161-1/+1
|/ / / / / / / /
* | | | | | | | Merge branch 'relay-platform-encode' into 'main'Ian Jackson2026-06-162-4/+21
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Encoding for RelayPlatform See merge request tpo/core/arti!4114
| * | | | | | | | tor-netdoc: Add ItemValueEncodable for RelayPlatformClara Engler2026-06-162-0/+14
| | | | | | | | |
| * | | | | | | | tor-netdoc: Implement ItemValueParseable instead for RelayPlatformClara Engler2026-06-162-4/+7
| | |_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ItemArgumentParseable does not make much sense because the field is effectively a free-form field similar to ContactInfo.
* | | | | | | | Merge branch 'edcert-papercuts' into 'main'Ian Jackson2026-06-161-38/+8
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix small edcert papercuts See merge request tpo/core/arti!4112
| * | | | | | | | tor-netdoc: Remove unequal key check from edcertsClara Engler2026-06-161-16/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As a follow up to !4022 and discussed on IRC: > Since we don't think we need to check the keys are different I think > it's OK to delete the thing in the tests that insists we have such a > check. > [...] > We are the relying party here. That MUST is directed to the signing > party. As reliers we don't need to check it.
| * | | | | | | | tor-netdoc: Merge edcert tests togetherClara Engler2026-06-161-22/+4
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes a TODO as discussed in !4022. Review with --color-moved.
* | | | | | | | Merge branch 'reconfig-check' into 'main'David Goulet2026-06-161-15/+42
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | arti-client: Warn if `reconfigure()` possibly left the client in an inconsistent state See merge request tpo/core/arti!4101
| * | | | | | | | arti-client: warn if reconfig left client in inconsistent stateSteven Engler2026-06-111-15/+42
| | | | | | | | |
* | | | | | | | | Merge branch 'flowctrl-tests' into 'main'opara2026-06-163-15/+435
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-proto: Add unit test for `XonXoffReader` See merge request tpo/core/arti!4093
| * | | | | | | | | tor-proto: add a unit test for `XonXoffReader`Steven Engler2026-06-162-1/+399
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The 'futures' version bump is needed so that our test can use `UnboundedSender::try_recv()` in the minimal-versions CI test.
| * | | | | | | | | tor-proto: add `DrainRateNotifier` trait for `XonXoffReader`Steven Engler2026-06-101-13/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will allow us to add a unit test for `XonXoffReader`.
| * | | | | | | | | tor-cell: impl `PartialEq + Eq` on `XonKbpsEwma`Steven Engler2026-06-101-1/+1
| | | | | | | | | |