| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| | * | | 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 | |||||
| | * | rpc-connect-sketch: clarify "no new critical members" principle. | Nick Mathewson | 2024-11-13 | 1 | -5/+11 | |
| | | | ||||||
| * | | Merge branch 'relay-bin-2' into 'main' | David Goulet | 2024-11-13 | 6 | -6/+216 | |
| |\ \ | |/ |/| | | | | | arti-relay: add clap for cli See merge request tpo/core/arti!2630 | |||||
| | * | arti-relay: add cli tests | Steven Engler | 2024-11-13 | 1 | -0/+35 | |
| | | | ||||||
| | * | arti-relay: add 'build-info' subcommand | Steven Engler | 2024-11-13 | 3 | -0/+57 | |
| | | | ||||||
| | * | arti-relay: add clap for cli | Steven Engler | 2024-11-13 | 4 | -2/+113 | |
| | | | | | | | | | The options are inspired by the arti cli interface. | |||||
| | * | arti-relay: add 'test-all-crates' and 'matrix-check' exceptions | Steven Engler | 2024-11-13 | 2 | -2/+7 | |
| | | | ||||||
| | * | arti-relay: add 'default-runtime' feature | Steven Engler | 2024-11-13 | 1 | -2/+4 | |
| |/ | ||||||
| * | Merge branch 'rpc-spec-updates' into 'main' | Nick Mathewson | 2024-11-13 | 2 | -8/+56 | |
| |\ | | | | | | | | | Miscellaneous RPC connect spec tweaks See merge request tpo/core/arti!2623 | |||||
| | * | rpc-connect-sketch: Per discussion, change non-localhost to "decline" | Nick Mathewson | 2024-11-12 | 1 | -1/+1 | |
| | | | ||||||
| | * | rpc-connect-sketc: Add question on TOML tolerance. | Nick Mathewson | 2024-11-07 | 1 | -0/+6 | |
| | | | ||||||
| | * | rpc-connect-sketch: more specific abort/decline documentation | Nick Mathewson | 2024-11-07 | 1 | -6/+12 | |
| | | | ||||||
| | * | rpc-connect-sketch: more on philosophy of client error handling | Nick Mathewson | 2024-11-07 | 1 | -2/+34 | |
| | | | ||||||
| | * | rpc-cookie: shell expansion happens before the addr is canonical | Nick Mathewson | 2024-11-07 | 1 | -0/+2 | |
| | | | ||||||
| | * | rpc-connect: Clarify server behavior on disabled entries. | Nick Mathewson | 2024-11-07 | 1 | -0/+2 | |
| | | | | | | | | | | | The behavior in question is to ignore the entries entirely, and not check their files at all. | |||||
| * | | Merge branch 'reconfigure' into 'main' | opara | 2024-11-13 | 1 | -2/+9 | |
| |\ \ | | | | | | | | | | | | | tor-chanmgr: fix reconfigure bug when given `CheckAllOrNothing` See merge request tpo/core/arti!2619 | |||||
| | * | | tor-chanmgr: fix reconfigure bug when given `CheckAllOrNothing` | Steven Engler | 2024-11-12 | 1 | -2/+9 | |
| |/ / | | | | | | | We shouldn't actually reconfigure anything if given `CheckAllOrNothing`. | |||||
