| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| | * | | | tor-config-path: Note an unused error variant | Nick Mathewson | 2024-11-19 | 1 | -0/+2 | |
| |/ / / | ||||||
| * | | | Merge branch 'config-paths' into 'main' | opara | 2024-11-18 | 17 | -213/+484 | |
| |\ \ \ | | | | | | | | | | | | | | | | | tor-config-path: refactor to support a `CfgPathResolver` See merge request tpo/core/arti!2615 | |||||
| | * | | | arti-client: change test to use path resolver from `TorClientConfig` | Steven Engler | 2024-11-18 | 1 | -6/+12 | |
| | | | | | ||||||
| | * | | | tor-config-path: add more documentation to `CfgPathResolver` | Steven Engler | 2024-11-18 | 1 | -0/+9 | |
| | | | | | ||||||
| | * | | | tor-config-path: remove `Option` from resolver value | Steven Engler | 2024-11-18 | 2 | -22/+28 | |
| | | | | | ||||||
| | * | | | arti-client: add path resolver to `TorClient` and `TorClientConfig` | Steven Engler | 2024-11-18 | 4 | -41/+70 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | Rather than using `arti_client::config::path_resolver()`, third-party code can get the path resolver using `TorClientConfig`s `AsRef<CfgPathResolver>` impl instead. | |||||
| | * | | | arti-client: add a more-comprehensive cfg variable test | Steven Engler | 2024-11-18 | 1 | -10/+35 | |
| | | | | | ||||||
| | * | | | arti-client: moved two tests from tor-config-path | Steven Engler | 2024-11-18 | 2 | -23/+20 | |
| | | | | | | | | | | | | | | | | | Also updated to use the `PATH_RESOLVER` resolver. | |||||
| | * | | | update `CfgPath::path` to use a `CfgPathResolver` | Steven Engler | 2024-11-18 | 13 | -144/+220 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a big change across multiple crates since there isn't a good way to break it up. This changes the signature of `CfgPath::path` to: ``` pub fn path(&self, path_resolver: &CfgPathResolver) -> Result<PathBuf, CfgPathError> { ``` Making this change means that our global `CfgPathResolver` needs to be stored in the 'arti-client' library instead of `tor-config-path`, and must be passed through to anything that calls `path` to expand the variables. | |||||
| | * | | | tor-ptmgr: pass through the `CfgPathResolver` | Steven Engler | 2024-11-18 | 3 | -3/+19 | |
| | | | | | ||||||
| | * | | | tor-hsservice: pass through the `CfgPathResolver` | Steven Engler | 2024-11-18 | 6 | -15/+55 | |
| | | | | | ||||||
| | * | | | tor-config-path: begin using the `CfgPathResolver` | Steven Engler | 2024-11-18 | 1 | -20/+28 | |
| | | | | | ||||||
| | * | | | tor-config-path: add unused `CfgPathResolver` | Steven Engler | 2024-11-18 | 1 | -1/+60 | |
| | | | | | ||||||
| | * | | | tor-config-path: moved 'impl Display for CfgPath' block | Steven Engler | 2024-11-18 | 1 | -9/+9 | |
| | | | | | ||||||
| | * | | | tor-config-path: moved two tests | Steven Engler | 2024-11-18 | 1 | -35/+35 | |
| |/ / / | | | | | | | | | | | | | These tests were in the 'test_serde' module, which wasn't the right place for them. | |||||
| * | | | Merge branch 'dummy-keymgr-docs' into 'main' | Ian Jackson | 2024-11-18 | 2 | -14/+30 | |
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | tor-keymgr: Explain why the dummy keymgr is missing some functions. Closes #1732 See merge request tpo/core/arti!2631 | |||||
| | * | | | tor-keymgr: Say that another option is to enable keymgr unconditionally. | Gabriela Moldovan | 2024-11-14 | 1 | -0/+2 | |
| | | | | | | | | | | | | | | | | | | | | | This commit applies Diziet's suggestion from https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2631#note_3130234 | |||||
| | * | | | tor-keymgr: Remove a couple of dummy KeyMgr functions. | Gabriela Moldovan | 2024-11-14 | 2 | -14/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are not actually needed anywhere. In fact, it's a bad idea to have them in the first place (see the comment from the dummy `KeyMgr` impl block). | |||||
| | * | | | tor-keymgr: Explain why the dummy keymgr is missing some functions. | Gabriela Moldovan | 2024-11-14 | 1 | -1/+28 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is based on a recent discussion with @Diziet on IRC. For the full context, see the chat log attached to the ticket. Closes #1732 | |||||
| * | | | | Merge branch 'relay-bin-2' into 'main' | opara | 2024-11-14 | 5 | -35/+9 | |
| |\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | arti-relay: remove runtime-related features Closes #1744 See merge request tpo/core/arti!2634 | |||||
| | * | | | arti-relay: small change to log level comment | Steven Engler | 2024-11-14 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | In arti this option is just a convenience function to override the configuration option, so we'll probably want to do the same thing. | |||||
| | * | | | arti-relay: remove TODO comment about runtimes | Steven Engler | 2024-11-14 | 1 | -7/+3 | |
| | | | | | ||||||
| | * | | | arti-relay: remove runtime/tls-related features | Steven Engler | 2024-11-14 | 3 | -27/+5 | |
| | |/ / | ||||||
| * | | | Merge branch 'keymgr-cert-note' into 'main' | gabi-250 | 2024-11-14 | 1 | -0/+412 | |
| |\ \ \ | | | | | | | | | | | | | | | | | doc/dev/notes: keymgr-certificates: Add note about certificate management. See merge request tpo/core/arti!2565 | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Clarify how keymgr might generate the cert. | Gabriela Moldovan | 2024-11-14 | 1 | -2/+3 | |
| | | | | | | | | | | | | | | | | | | | | | This is not necessarily set in stone, but it seems like a plausible way of going about this. | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Emphasize that we won't manage arbitrary ↵ | Gabriela Moldovan | 2024-11-14 | 1 | -3/+8 | |
| | | | | | | | | | | | | | | | | | certs. | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Abolish get_cert_entry. | Gabriela Moldovan | 2024-11-14 | 1 | -16/+0 | |
| | | | | | | | | | | | | | | | | | This also removes the reference to the `get_entry()` function. | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Allow the cert denotators to be absent. | Gabriela Moldovan | 2024-11-14 | 1 | -11/+9 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Update proposed renaming. | Gabriela Moldovan | 2024-11-14 | 1 | -28/+19 | |
| | | | | | | | | | | | | | | | | | | | | | Based on our discussion in https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2565#note_3129065 | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Apply key format clarifications from Diziet. | Gabriela Moldovan | 2024-11-14 | 1 | -0/+7 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Apply suggestions from Diziet. | gabi-250 | 2024-11-14 | 1 | -2/+2 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Clarify code docs. | Gabriela Moldovan | 2024-11-14 | 1 | -4/+8 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Fix typo. | Gabriela Moldovan | 2024-11-14 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Remove unreadable chunk from intro. | Gabriela Moldovan | 2024-11-14 | 1 | -8/+3 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Remove unnecessary args. | Gabriela Moldovan | 2024-11-14 | 1 | -4/+0 | |
| | | | | | | | | | | | | | | | | | | | | | We don't actually need a `KeyType` argument (`KeyTypes` are obtained from the `EncodableKey::key_type()` impl of the key/cert). | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Reorder args for clarity. | Gabriela Moldovan | 2024-11-14 | 1 | -4/+4 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Move subject key specifier to trait. | Gabriela Moldovan | 2024-11-14 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | This helps us cut down on the number of args in `get_*_and_cert()`. | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Say that cert paths are derived from key ↵ | Gabriela Moldovan | 2024-11-14 | 1 | -26/+39 | |
| | | | | | | | | | | | | | | | | | paths. | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Mention the uid we settled on in !2577. | Gabriela Moldovan | 2024-11-14 | 1 | -5/+5 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Remove outdated design. | Gabriela Moldovan | 2024-11-14 | 1 | -191/+33 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We decided to go with the second proposed design, so we can safely remove this one now. This also updates the introduction to summarize how we arrived at this design. | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Clarify confusing wording. | Gabriela Moldovan | 2024-11-14 | 1 | -3/+2 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Make the signing_key optionally present ↵ | Gabriela Moldovan | 2024-11-14 | 1 | -3/+37 | |
| | | | | | | | | | | | | | | | | | in keystore. | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Remove redundant fields from ↵ | Gabriela Moldovan | 2024-11-14 | 1 | -5/+0 | |
| | | | | | | | | | | | | | | | | | KeyCertificateSpecifier. | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Clarify what KeyType/CertType is about. | Gabriela Moldovan | 2024-11-14 | 1 | -0/+7 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Remove misleading statement. | Gabriela Moldovan | 2024-11-14 | 1 | -1/+0 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Add a revised proposal. | Gabriela Moldovan | 2024-11-14 | 1 | -0/+358 | |
| | | | | | | | | | | | | | | | | | | | | | Based on Diziet's feedback in https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2565#note_3099397 | |||||
| | * | | | doc/dev/notes: keymgr-certificates: Say that the original proposal is rejected. | Gabriela Moldovan | 2024-11-14 | 1 | -7/+12 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Update key notation for clarity. | Gabriela Moldovan | 2024-11-14 | 1 | -6/+8 | |
| | | | | | ||||||
| | * | | | doc/dev/notes: keymgr-certificates: Add note about certificate management. | Gabriela Moldovan | 2024-11-14 | 1 | -0/+177 | |
| |/ / / | | | | | | | | | | | | | | | | | | | This describes the storage format and on-disk key store paths of `relaysign_ed` certificates (which we'll soon need to support). Part of #1617 | |||||
| * | | | Merge branch 'rpc-spec-updates-2' into 'main' | Ian Jackson | 2024-11-14 | 1 | -5/+11 | |
| |\ \ \ | |/ / |/| | | | | | | | | rpc-connect-sketch: clarify "no new critical members" principle. See merge request tpo/core/arti!2628 | |||||
 |
index : mirrors/arti.git | |
| mirror of https://gitlab.torproject.org/tpo/core/arti |
| summaryrefslogtreecommitdiff |