| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| | * | | tor-keymgr: path errors: Add some xrefs | Ian Jackson | 2023-12-07 | 2 | -1/+9 | |
| | | | | ||||||
| | * | | tor-keymgr: ArtiPathSyntaxError: rename from ArtiPathError (fmt) | Ian Jackson | 2023-12-07 | 3 | -12/+45 | |
| | | | | ||||||
| | * | | tor-keymgr: ArtiPathSyntaxError: rename from ArtiPathError | Ian Jackson | 2023-12-07 | 7 | -41/+40 | |
| | | | | | | | | | | | | | | | | The previous name was ambiguous, and confusable with KeyPathError (of which it is, basically, a variant). | |||||
| | * | | tor-keymgr: Abolish unneeded StdResult alias in arti_path | Ian Jackson | 2023-12-07 | 1 | -6/+4 | |
| | | | | ||||||
| | * | | tor-keymgr: Move ArtiPath etc. into new module (docs links) | Ian Jackson | 2023-12-07 | 1 | -2/+6 | |
| | | | | | | | | | | | | | Fix docs link breakage. | |||||
| | * | | tor-keymgr: Move ArtiPath etc. into new module | Ian Jackson | 2023-12-07 | 3 | -191/+200 | |
| | | | | | | | | | | | | | Minimal changes to still compile and pass tests. | |||||
| | * | | tor-keymgr: KeySpecifierComponent::to_component: rename from as_component | Ian Jackson | 2023-12-07 | 1 | -7/+7 | |
| | | | | | | | | | | | | | | | | | | | Normally as_... is a cheap and non-consuming (reference to reference) conversion. to_ is better here since we don't consume, but we do allocate. | |||||
| | * | | 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 (fmt) | Ian Jackson | 2023-12-07 | 1 | -3/+8 | |
| | | | | ||||||
| | * | | tor-keymgr: Make KeySpecifierComponent::as_component fallible | Ian Jackson | 2023-12-07 | 2 | -29/+36 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 | 2 | -11/+9 | |
| | | | | ||||||
| | * | | tor-keymgr: Introduce InvalidKeyPathComponentValue errors | Ian Jackson | 2023-12-07 | 3 | -15/+74 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | 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 | 2 | -7/+7 | |
| | | | | | | | | | | | | | | | | Call sites are shortly going to need the component on the error path, so this function ought probably not to consume it. | |||||
| | * | | tor-keymgr: Make two errors not be Copy | Ian Jackson | 2023-12-07 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | This is a hostage to fortune. (In a previous version of !1796 I wanted to make it untrue.) | |||||
| | * | | tor-keymgr: KeySpecifierDefault: role: allow runtime value | Ian Jackson | 2023-12-07 | 1 | -15/+72 | |
| | | | | | | | | | | | | | Make it possible for the role to be a field. | |||||
| | * | | tor-keymgr: KeySpecifierDefault: Use meta as str | Ian Jackson | 2023-12-07 | 1 | -3/+6 | |
| | | | | | | | | | | | | | This saves on some stringify! and improves error checking. | |||||
| | * | | tor-keymgr: KeySpecifierDefault: fixed_path_component feature | Ian Jackson | 2023-12-07 | 1 | -0/+29 | |
| | | | | ||||||
| | * | | tor-keymgr: KeySpecifierDefault: recast field loop | Ian Jackson | 2023-12-07 | 1 | -1/+2 | |
| | | | | | | | | | | | | | | | | We're going to do another per-field thing that doesn't want this condition. | |||||
| | * | | tor-keymgr: KeySpecifierDefault: ctor_path feature | Ian Jackson | 2023-12-07 | 1 | -8/+47 | |
| | | | | ||||||
| | * | | tor-keymgr: KeySpecifierDefault: Document attributes more formally | Ian Jackson | 2023-12-07 | 1 | -1/+20 | |
| | | | | | | | | | | | | | | | | | | | The `summary` attribute refers to a field in KeyPathInfo but this field isn't public. We can refer to builder, but there should be getters or something. | |||||
| | * | | tor-keymgr: KeySpecifierDefault: Move dead code allow | Ian Jackson | 2023-12-07 | 1 | -5/+1 | |
| | | | | ||||||
| | * | | tor-keymgr: KeyPathInfo: Provide builder() method | Ian Jackson | 2023-12-07 | 1 | -4/+11 | |
| | | | | | | | | | | | | | Otherwise the KeyPathInfo rustdocs don't have a method for making one. | |||||
| | * | | tor-keymgr: Replace assert_key_specifier_rountrip macro with function (fmt) | Ian Jackson | 2023-12-07 | 2 | -26/+8 | |
| | | | | | | | | | | | | | Formatting churn at call sites. | |||||
| | * | | tor-keymgr: Replace assert_key_specifier_rountrip macro with function | Ian Jackson | 2023-12-07 | 3 | -45/+50 | |
| | | | | | | | | | | | | | The new function checks both directions. The (misspelled) macro didn't. | |||||
| | * | | tor-keymgr: provide KeySpecifierComponentViaDisplayFromStr and use it | Ian Jackson | 2023-12-07 | 3 | -39/+24 | |
| | | | | | | | | | | | | | Now the only manual impl is TimePeriod, which is indeed special. | |||||
| | * | | ArtiPaths: derive FromStr for ArtiPath ArtiPathComponent HsNickname (fmt) | Ian Jackson | 2023-12-07 | 2 | -4/+27 | |
| | | | | | | | | | | | | | Mandatory horrible formatting. | |||||
| | * | | ArtiPaths: derive FromStr for ArtiPath ArtiPathComponent HsNickname | Ian Jackson | 2023-12-07 | 2 | -0/+3 | |
| | | | | ||||||
| * | | | Merge branch 'hss_status' into 'main' | Nick Mathewson | 2023-12-07 | 5 | -13/+184 | |
| |\ \ \ | | | | | | | | | | | | | | | | | Add a status API to OnionService. See merge request tpo/core/arti!1797 | |||||
| | * | | | 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. | |||||
| * | | | Merge branch 'new-toml' into 'main' | Nick Mathewson | 2023-12-07 | 9 | -9/+11 | |
| |\ \ \ | |/ / |/| | | | | | | | | upgrade to toml 0.8.8 See merge request tpo/core/arti!1794 | |||||
| | * | | upgrade to toml 0.8.8 | Nick Mathewson | 2023-12-05 | 9 | -9/+11 | |
| | |/ | ||||||
| * | | Merge branch 'warn' into 'main' | Ian Jackson | 2023-12-06 | 2 | -5/+6 | |
| |\ \ | | | | | | | | | | | | | tor-keymgr: Fix some docs warnings See merge request tpo/core/arti!1803 | |||||
| | * | | tor-keymgr: Fix some docs warnings | Ian Jackson | 2023-12-06 | 2 | -5/+6 | |
| | | | | | | | | | | | | | | | | Fixes cargo doc --locked --workspace --all-features --document-private-items | |||||
| * | | | 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-keymgr: Update wording in TODO. | Gabriela Moldovan | 2023-12-05 | 1 | -1/+1 | |
| | | | ||||||
| * | | tor-hsservice: Add a cross-reference to KeystoreSweeper. | Gabriela Moldovan | 2023-12-05 | 1 | -0/+7 | |
| | | | ||||||
| * | | tor-keymgr: Call inventory::collect in the dummy keymgr too. | Gabriela Moldovan | 2023-12-05 | 1 | -0/+2 | |
| | | | | | | | | | | | Not calling it results in an error (it instantiates the plugin registry for our type). | |||||
