aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | 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
| | | | |
| * | | | arti-client: Fix formattinghjrgrn2026-07-102-9/+2
| | | | |
| * | | | tor-persist: Fix clippy warninghjrgrn2026-07-101-1/+1
| | | | |
| * | | | multiple crates: Fix clippy warningshjrgrn2026-07-1023-42/+42
| | | | |
| * | | | fs-mistrust: Fix clippy warningshjrgrn2026-07-101-1/+1
| | | | |
| * | | | safelog: Fix clippy warningshjrgrn2026-07-101-5/+5
| | | | |
| * | | | maint: fixup-features: Fix clippy warningshjrgrn2026-07-101-4/+4
| | | | |
| * | | | test-temp-dir: Fix clippy warninghjrgrn2026-07-101-2/+2
| | | | |
| * | | | tor-error: Fix clippy warningshjrgrn2026-07-102-2/+2
| | | | |
| * | | | retry-error: Fix clippy warninghjrgrn2026-07-101-1/+1
| | |/ / | |/| |
* | | | Merge branch 'upgrade-cipher' into 'main'gabi-2502026-07-1311-61/+126
|\ \ \ \ | |/ / / |/| | | | | | | | | | | Upgrade cipher, aes, and ctr. See merge request tpo/core/arti!4195
| * | | Upgrade cipher, aes, and ctr.Nick Mathewson2026-07-1311-61/+126
| | | |