| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | | tor-hsservice: Trim trailing whitespace. | Gabriela Moldovan | 2023-12-11 | 1 | -1/+1 | |
| |/ | ||||||
| * | tor-hsservice: Remove unused lifetime. | Gabriela Moldovan | 2023-12-07 | 1 | -1/+1 | |
| | | | | | Fixes a clippy lint. | |||||
| * | tor-hsservice: Remove unused function. | Gabriela Moldovan | 2023-12-07 | 1 | -16/+0 | |
| | | ||||||
| * | tor-hsservice: Fix publisher bug causing unnecessary uploads. | Gabriela Moldovan | 2023-12-07 | 1 | -27/+31 | |
| | | | | | | | | This fixes a bug where the publisher wasn't preserving the `DescriptorStatus` of its `HsDirs` when handling consensus changes. The bug is described in more detailed in the TODO removed by this commit. | |||||
| * | tor-hsservice: Add a TODO about a publlisher bug | Gabriela Moldovan | 2023-12-07 | 1 | -0/+17 | |
| | | ||||||
| * | tor-hsservice: Rename period to ctx for clarity. | Gabriela Moldovan | 2023-12-07 | 1 | -2/+2 | |
| | | | | | This is actually a `TimePeriodContext`. | |||||
| * | Merge branch 'persist-macro' into 'main' | Ian Jackson | 2023-12-07 | 6 | -96/+78 | |
| |\ | | | | | | | | | | | | | Improve KeySpecifier, errors, IptKeySpecifier impl, etc. Closes #1116 and #1148 See merge request tpo/core/arti!1796 | |||||
| | * | tor-hsservice: keys: Remove unused imports (and a blocking todo) | Ian Jackson | 2023-12-07 | 1 | -2/+0 | |
| | | | ||||||
| | * | tor-hsservice: tests: IptKeySpecifier: Do round trip test | Ian Jackson | 2023-12-07 | 1 | -8/+8 | |
| | | | ||||||
| | * | tor-hsservice: IptKeySpecifier: impl KeySpecifier via macro | Ian Jackson | 2023-12-07 | 1 | -14/+8 | |
| | | | ||||||
| | * | tor-hsservice: IptKeySpecifier: make it owned | Ian Jackson | 2023-12-07 | 2 | -5/+5 | |
| | | | | | | | | | | | The macro is going to want to be able to do a back-conversion from an ArtiPath, so there has to be an owned version. | |||||
| | * | tor-hsservice: IptKeySpecifier: reorder fields | Ian Jackson | 2023-12-07 | 1 | -2/+2 | |
| | | | | | | | | | | | Now they are in the order they appear in the ArtiPath, which is a bit clearer. | |||||
| | * | tor-hsservice: IptKeyRole: impl FromStr and KeySpecifierComponent | Ian Jackson | 2023-12-07 | 1 | -1/+4 | |
| | | | ||||||
| | * | tor-hsservice: IptLocalId: impl KeySpecifierComponent | Ian Jackson | 2023-12-07 | 1 | -0/+3 | |
| | | | ||||||
| | * | tor-hsservice: IptLocalId: impl FromStr (fmt) | Ian Jackson | 2023-12-07 | 1 | -4/+2 | |
| | | | ||||||
| | * | tor-hsservice: IptLocalId: impl FromStr | Ian Jackson | 2023-12-07 | 2 | -27/+18 | |
| | | | | | | | | | And implement serde that way, using SerdeStringOrTransparent. | |||||
| | * | tor-hsservice: Expose SerdeStringOrTransparent to crate | Ian Jackson | 2023-12-07 | 2 | -2/+4 | |
| | | | | | | | | | | | And make it useable elsewhere by not expecting the particular error type. | |||||
| | * | tor-keymgr: Make KeySpecifierComponent::as_component fallible | Ian Jackson | 2023-12-07 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | This allows a principled implentation via Display. The alternative would be to have runtime panics whenever someone uses KeySpecifierComponentViaDisplayFromStr but the Display impl can produce wrong syntax (that doesn't fit in an ArtiPathComponent). Bug seems like the right error since implemnting KeySpecifierComponent implies that every value is representable in an ArtiPathComponent. | |||||
| | * | tor-keymgr: Introduce InvalidKeyPathComponentValue errors (fmt) | Ian Jackson | 2023-12-07 | 1 | -9/+8 | |
| | | | ||||||
| | * | tor-keymgr: Introduce InvalidKeyPathComponentValue errors | Ian Jackson | 2023-12-07 | 1 | -1/+7 | |
| | | | | | | | | | | | | | | | | | This is a variant in KeyPathError, which contains the actual wrong value, and the key name. But it is also a unit struct to return from the trait impl. This replaces ArtiPathError::InvalidDenotator. | |||||
| | * | tor-hsservice: Prepare for different denotator error handling | Ian Jackson | 2023-12-07 | 1 | -5/+6 | |
| | | | | | | | | | | | Save the ArtiPathComponent in a variable and separate out the Ok(tp) for clarity. No functional change. | |||||
| | * | tor-keymgr: Have KeySpecifierComponent::from_component take a ref | Ian Jackson | 2023-12-07 | 1 | -1/+1 | |
| | | | | | | | | | | | Call sites are shortly going to need the component on the error path, so this function ought probably not to consume it. | |||||
| | * | tor-keymgr: Replace assert_key_specifier_rountrip macro with function (fmt) | Ian Jackson | 2023-12-07 | 1 | -12/+3 | |
| | | | | | | | | | Formatting churn at call sites. | |||||
| | * | tor-keymgr: Replace assert_key_specifier_rountrip macro with function | Ian Jackson | 2023-12-07 | 1 | -13/+8 | |
| | | | | | | | | | The new function checks both directions. The (misspelled) macro didn't. | |||||
| | * | tor-keymgr: provide KeySpecifierComponentViaDisplayFromStr and use it | Ian Jackson | 2023-12-07 | 1 | -13/+2 | |
| | | | | | | | | | Now the only manual impl is TimePeriod, which is indeed special. | |||||
| | * | ArtiPaths: derive FromStr for ArtiPath ArtiPathComponent HsNickname (fmt) | Ian Jackson | 2023-12-07 | 1 | -2/+13 | |
| | | | | | | | | | Mandatory horrible formatting. | |||||
| | * | ArtiPaths: derive FromStr for ArtiPath ArtiPathComponent HsNickname | Ian Jackson | 2023-12-07 | 1 | -0/+1 | |
| | | | ||||||
| * | | Add a "Shutdown" status, and clarify places where status should change. | Nick Mathewson | 2023-12-07 | 4 | -5/+19 | |
| | | | ||||||
| * | | Add a status API to OnionService. | Nick Mathewson | 2023-12-07 | 3 | -11/+168 | |
| |/ | | | | | | (This doesn't yet send any status events.) Begins work on arti#1083. | |||||
| * | tor-hsservice: Add more info to an internal error (fmt). | Gabriela Moldovan | 2023-12-05 | 1 | -6/+7 | |
| | | ||||||
| * | tor-hsservice: Add more info to an internal error. | Gabriela Moldovan | 2023-12-05 | 1 | -1/+9 | |
| | | | | | | Logging the current time and the start of the time period might help us debug #1143 (we'll at least know how far off `now` is from `start`). | |||||
| * | tor-hsservice: Fix clippy warning. | Gabriela Moldovan | 2023-12-05 | 1 | -1/+1 | |
| | | ||||||
| * | tor-hsservice: Wrap some long lines in a helper macro. | Gabriela Moldovan | 2023-12-05 | 1 | -4/+11 | |
| | | ||||||
| * | tor-hsservice: Add a cross-reference to KeystoreSweeper. | Gabriela Moldovan | 2023-12-05 | 1 | -0/+7 | |
| | | ||||||
| * | tor-keymgr: Abolish the KeyDenotator trait in favour of KeyPathComponent (fmt). | Gabriela Moldovan | 2023-12-05 | 1 | -2/+6 | |
| | | ||||||
| * | tor-keymgr: Abolish the KeyDenotator trait in favour of KeyPathComponent. | Gabriela Moldovan | 2023-12-05 | 1 | -2/+4 | |
| | | ||||||
| * | tor-hsservice: Stop the KeystoreSweeper task when the service exits (fmt). | Gabriela Moldovan | 2023-12-05 | 3 | -57/+62 | |
| | | ||||||
| * | tor-hsservice: Stop the KeystoreSweeper task when the service exits. | Gabriela Moldovan | 2023-12-05 | 2 | -7/+30 | |
| | | ||||||
| * | tor-hsservice: Make shutdown a broadcast channel. | Gabriela Moldovan | 2023-12-05 | 2 | -11/+13 | |
| | | | | | | We will soon need another component to be able to subscribe to the shutdown signal. | |||||
| * | tor-hsservice: Move remove_if_expired macro inside task (fmt). | Gabriela Moldovan | 2023-12-05 | 1 | -9/+3 | |
| | | ||||||
| * | tor-hsservice: Move remove_if_expired macro inside task. | Gabriela Moldovan | 2023-12-05 | 1 | -37/+22 | |
| | | ||||||
| * | tor-keymgr: Update key summary strings to be less verbose. | Gabriela Moldovan | 2023-12-05 | 1 | -5/+5 | |
| | | ||||||
| * | tor-hsservice: Test the TryFrom<&KeyPath> KeySpecifier impl. | Gabriela Moldovan | 2023-12-05 | 2 | -1/+8 | |
| | | ||||||
| * | tor-hsservice: Add some useful derives for the key specifiers. | Gabriela Moldovan | 2023-12-05 | 1 | -5/+5 | |
| | | ||||||
| * | tor-hsservice: Add a helper for removing expired keys. | Gabriela Moldovan | 2023-12-05 | 2 | -3/+154 | |
| | | | | | Closes #1043 | |||||
| * | tor-hsservice: Make the KeySpecifier fields crate-public. | Gabriela Moldovan | 2023-12-05 | 1 | -7/+7 | |
| | | | | | We're about to use them. | |||||
| * | tor-keymgr: Store an owned type instead. | Gabriela Moldovan | 2023-12-05 | 6 | -43/+52 | |
| | | | | | | | We need this because the key specifier types will soon be owned types too (they will soon grow a From<&KeyPath> impl). We _could_ make them store `Cow`s instead, but I think that's a premature optimization. | |||||
| * | tor-keymgr: Make all arguments of arti_pattern optional. | Gabriela Moldovan | 2023-12-05 | 1 | -1/+1 | |
| | | | | | | We're going to need this soon (to match all hidden service keys, regardless of nickname, for example). | |||||
| * | tor-keymgr: Introduce a KeySpecifierComponent trait. | Gabriela Moldovan | 2023-12-05 | 1 | -1/+19 | |
| | | | | | | | | | | Now the fields of a `DefaultKeySpecifier` no longer have to implement `Display` (they need to implement `KeySpecifierComponent`). See https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1733#note_2966400 Closes #1127 | |||||
| * | tor-keymgr: Change KeyDenotator error type to KeyPathError. | Gabriela Moldovan | 2023-12-05 | 1 | -2/+2 | |
| | | | | | | This is a separate class of errors that deserves its own error type. We will soon use `KeyPathError` in other contexts as well. | |||||
