aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | Merge branch 'remove-lint' into 'main'Jim Newsome2026-07-15114-162/+77
|\ \ | | | | | | | | | | | | | | | | | | Removed unnecessary lint Closes #2556 See merge request tpo/core/arti!4210
| * | Remove now-unneeded allow(clippy::cognitive_complexity)Jim Newsome2026-07-1543-75/+0
| | |
| * | add_warning: add reference to arti#2556Jim Newsome2026-07-1574-74/+74
| | |
| * | lib.rs: remove stale commentJim Newsome2026-07-151-2/+0
| | |
| * | Removed unnecessary lintpryty262026-07-1583-85/+77
| | | | | | | | | | | | Removed unnecessary lint
* | | Merge branch 'consensus-method-type' into 'main'Ian Jackson2026-07-159-1/+159
|\ \ \ | | | | | | | | | | | | | | | | dirauth: Supported consensus method type See merge request tpo/core/arti!4200
| * | | tor-dirauth: Move supported_well_formed test case to consensus.rsIan Jackson2026-07-152-10/+29
| | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4200#note_3433115
| * | | tor-dirauth: Add spec link to consensus methods listIan Jackson2026-07-151-0/+6
| | | | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4200#note_3433106
| * | | tor-dirauth: Replace map_ranges with local import and value wrappingIan Jackson2026-07-152-20/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4200#note_3433105 Suggested-by: Jim Newsome <[email protected]>
| * | | tor-dirauth: Introduce SupportedConsensusMethod newtypeIan Jackson2026-07-157-0/+142
| | | |
| * | | tor-netdoc: Expose ConsensusMethod's fieldIan Jackson2026-07-152-1/+3
|/ / /
* | | Merge branch 'ip-summary-precise' into 'main'Ian Jackson2026-07-155-0/+601
|\ \ \ | | | | | | | | | | | | | | | | Implement precise port summaries See merge request tpo/core/arti!4198
| * | | tor-netdoc: document port summaries in semver.mdIan Jackson2026-07-151-0/+1
| | | |
| * | | tor-netdoc: Test that test helper can fail as expectedIan Jackson2026-07-151-6/+39
| | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4198#note_3434449
| * | | tor-netdoc: Fix typos in docsIan Jackson2026-07-151-2/+2
| | | |
| * | | tor-netdoc: Implement precise port summariesIan Jackson2026-07-154-0/+567
|/ / / | | | | | | | | | | | | | | | | | | | | | For calculating `ipv6-policy` in routerdescs. Also implement for v4 for the benefit of future tests of approximate port policy summaries, for possible future protocol change, and for completeness.
* | | Merge branch 'da-relayid' into 'main'Ian Jackson2026-07-151-145/+119
|\ \ \ | | | | | | | | | | | | | | | | tor-linkspec: Use deftly to reduce copypasta in RelayId and RelayIdRef See merge request tpo/core/arti!4196
| * | | tor-linkspec: RelayIdType: Clarify display vs display_idIan Jackson2026-07-151-4/+4
| | | | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/4196#note_3433090
| * | | tor-linkspec: Un-explode and tidy some derive listsIan Jackson2026-07-011-17/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rustfmt likes this IMO-awful layout for derives. We can force it to let us manually break the line, if we use //, and then it stays put. No functional change.
| * | | tor-linkspec: Sort out indentation in new macro templateIan Jackson2026-07-011-111/+111
| | | | | | | | | | | | | | | | Pure whitespace change. Review with, eg, git diff -w.
| * | | tor-linkspec: PartialEq: use deftly, not an ad-hoc macro_rulesIan Jackson2026-07-011-10/+2
| | | | | | | | | | | | | | | | This gets rid of the matcher, and the need to invoke it.
| * | | tor-linkspec: PartialEq: change variable namesIan Jackson2026-07-011-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change $var to $vname, and $type to $IDENTITY. These are the names used in the deftly template. This will make the next commit almost a no-op.
| * | | tor-linkspec: PartialEq: simplify PartialEq<FooIdentity> for RelayIdIan Jackson2026-07-011-1/+1
| | | | | | | | | | | | | | | | | | | | We can just refer to the PartialEq<FooIdentity> for RelayIdRef, which is macro-generated just above, replacing this copy-pasted matches.
| * | | tor-linkspec: Use deftly for RelayIdId and RelayIdRef implsIan Jackson2026-07-011-44/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes quite a lot of in-detail copy-paste. The lone `}` is being moved, bringing the code we're changing within the scope of the template.
| * | | tor-linkspec: Minimal deftly for RelayIdId and RelayIdRefIan Jackson2026-07-011-11/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Derive the two types. We end up moving the display strings from direct attributes directed to derive_more, to deftly attributes. Indentation is rather odd now: we ought to indent the macro body. We'll sort that out at the end. No functional change.
| * | | tor-linkspec: Move PartialEq implsIan Jackson2026-07-011-19/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Pure code motion. We want to move this to where a bigger macro is going to be, so that it can be part of it. Code motion only, review with --color-moved.
* | | | Merge branch 'nd' into 'main'Ian Jackson2026-07-1522-522/+778
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | tor-netdoc: Stabilise votes and encoding of authcerts (+ more tests) See merge request tpo/core/arti!4189
| * | | | tor-netdoc: Stabilise votes and encoding of authcerts (formatting)Ian Jackson2026-07-153-13/+8
| | | | |
| * | | | tor-netdoc: Stabilise votes and encoding of authcertsIan Jackson2026-07-159-37/+17
| | | | |
| * | | | tor-netdoc: Remove a now-unnecessary allowIan Jackson2026-07-151-1/+0
| | | | |
| * | | | tor-netdoc: drop a done TODOIan Jackson2026-07-151-3/+0
| | | | | | | | | | | | | | | | | | | | torspec!499 is merged.
| * | | | tor-netdoc tests: Test consensus verification error casesIan Jackson2026-07-153-5/+104
| | | | |
| * | | | tor-netdoc tests: Break out prep_netstatus_verify (fmt)Ian Jackson2026-07-151-6/+1
| | | | |
| * | | | tor-netdoc tests: Break out prep_netstatus_verifyIan Jackson2026-07-151-18/+32
| | | | |
| * | | | tor-netdoc tests: Test roundtrip of subset of live dataIan Jackson2026-07-152-2/+55
| | | | | | | | | | | | | | | | | | | | With facility to test roundtrip of whole consensus.
| * | | | tor-netdoc tests: Run new testdata-live-download scriptIan Jackson2026-07-153-0/+473
| | | | |
| * | | | tor-netdoc tests: Provide new testdata-live-download scriptIan Jackson2026-07-153-0/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I have confimed the consent of the operators of the relays listed in the script. The rationale for using .entire.FOO filenames for the curl outputs will become clear shortly.
| * | | | tor-netdoc: Add some missing "retain-unknown" cfg gates to testsIan Jackson2026-07-151-4/+4
| | | | | | | | | | | | | | | | | | | | Nothing in CI tests incomplete without retain-unknown, apparently.
| * | | | tor-netdoc: Now that poc is gone, make some things no longer pub(crate)Ian Jackson2026-07-151-9/+4
| | | | |
| * | | | tor-netdoc: Abolish pocIan Jackson2026-07-155-421/+0
| | | | |
| * | | | tor-dirserver: Use "plain" consensus terminology rather than "ns" (fmt)Ian Jackson2026-07-151-1/+4
| | | | |
| * | | | tor-dirserver: Use "plain" consensus terminology rather than "ns"Ian Jackson2026-07-151-12/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It doesn't make sense to say that a plain consensus is an "ns" consensus, because "ns" stands for "network status" and all consensus flavours, and indeed votes, are network statusus. That's why tor-netdoc now uses "plain". Use that here too.
| * | | | tor-dirserver: Use real consensus types, not poc (fmt)Ian Jackson2026-07-151-4/+1
| | | | |
| * | | | tor-dirserver: Use real consensus types, not pocIan Jackson2026-07-151-3/+2
|/ / / /
* | | | Merge branch 'spin' into 'main'gabi-2502026-07-151-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Update spin to 0.9.8 See merge request tpo/core/arti!4208
| * | | | Update spin to 0.9.8Ian Jackson2026-07-151-2/+2
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes cargo-audit failure, due to spin 0.9.8 being yanked: The changelog entry for 0.9.9 says: > ### Fixed > > - Unsoundness in `Once::force_into_inner`, `Once::try_into_inner`, and > `Once::into_inner_unchecked`. \[...] In our stack, spin <- lazy_static 1.5.0 (only). lazy_static 1.5.0 (git tag) has no hits for `into_inner`. So I think we don't have any calls to the previously-unsound methods.
* | | | Merge branch 'crossbeam-epoch' into 'main'Jim Newsome2026-07-131-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | maint/cargo-audit: Bump crossbeam-epoch to address RUSTSEC-2026-0204 See merge request tpo/core/arti!4207
| * | | | maint/cargo-audit: Bump crossbeam-epoch to address RUSTSEC-2026-0204Gabriela Moldovan2026-07-131-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | RUSTSEC-2026-0204 doesn't affect Arti, so this won't need a TROVE. I checked using the procedure documented under "Checking whether we are affected by a RUSTSEC" in `doc/dev/SecurityResponse.md`. I applied the following patch to my local `crossbeam-epoch` to remove the affected impls, and confirmed that Arti still builds successfully with it: ```diff diff --git a/crossbeam-epoch/src/atomic.rs b/crossbeam-epoch/src/atomic.rs index 41b4cd91..8b9b3a01 100644 --- a/crossbeam-epoch/src/atomic.rs +++ b/crossbeam-epoch/src/atomic.rs @@ -939,14 +939,6 @@ impl<T: ?Sized + Pointable> fmt::Debug for Atomic<T> { } } -impl<T: ?Sized + Pointable> fmt::Pointer for Atomic<T> { - fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result { - let data = self.data.load(Ordering::SeqCst); - let (raw, _) = decompose_tag::<T>(data); - fmt::Pointer::fmt(&(unsafe { T::deref(raw) as *const _ }), f) - } -} - impl<T: ?Sized + Pointable> Clone for Atomic<T> { /// Returns a copy of the atomic value. /// @@ -1660,12 +1652,6 @@ impl<T: ?Sized + Pointable> fmt::Debug for Shared<'_, T> { } } -impl<T: ?Sized + Pointable> fmt::Pointer for Shared<'_, T> { - fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result { - fmt::Pointer::fmt(&(unsafe { self.deref() as *const _ }), f) - } -} - impl<T: ?Sized + Pointable> Default for Shared<'_, T> { fn default() -> Self { Shared::null() ``` Note: the reason the `fmt::Pointer` impls I removed above look slightly different from the ones in the [crossbeam-rs MR] from the RUSTSEC page is because my patched `crossbeam-epoch` is based on the `crossbeam-epoch-0.9.18` tag rather than `main`. [crossbeam-rs MR]: https://github.com/crossbeam-rs/crossbeam/pull/1273/files
* | | | | Merge branch 'clippy_warning' into 'main'Jim Newsome2026-07-1331-64/+57
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | multiple crates: Fix clippy warnings See merge request tpo/core/arti!4203
| * | | | arti: subcommands: keys: Fix clippy::for-kv-maphjrgrn2026-07-131-2/+2
| | | | |