summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | tor-keymgr: Say that another option is to enable keymgr unconditionally.Gabriela Moldovan2024-11-141-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 Moldovan2024-11-142-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 Moldovan2024-11-141-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'opara2024-11-145-35/+9
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | arti-relay: remove runtime-related features Closes #1744 See merge request tpo/core/arti!2634
| * | arti-relay: small change to log level commentSteven Engler2024-11-141-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 runtimesSteven Engler2024-11-141-7/+3
| | |
| * | arti-relay: remove runtime/tls-related featuresSteven Engler2024-11-143-27/+5
| |/
* | Merge branch 'keymgr-cert-note' into 'main'gabi-2502024-11-141-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 Moldovan2024-11-141-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 Moldovan2024-11-141-3/+8
| | | | | | | | | | | | certs.
| * | doc/dev/notes: keymgr-certificates: Abolish get_cert_entry.Gabriela Moldovan2024-11-141-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 Moldovan2024-11-141-11/+9
| | |
| * | doc/dev/notes: keymgr-certificates: Update proposed renaming.Gabriela Moldovan2024-11-141-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 Moldovan2024-11-141-0/+7
| | |
| * | doc/dev/notes: keymgr-certificates: Apply suggestions from Diziet.gabi-2502024-11-141-2/+2
| | |
| * | doc/dev/notes: keymgr-certificates: Clarify code docs.Gabriela Moldovan2024-11-141-4/+8
| | |
| * | doc/dev/notes: keymgr-certificates: Fix typo.Gabriela Moldovan2024-11-141-1/+1
| | |
| * | doc/dev/notes: keymgr-certificates: Remove unreadable chunk from intro.Gabriela Moldovan2024-11-141-8/+3
| | |
| * | doc/dev/notes: keymgr-certificates: Remove unnecessary args.Gabriela Moldovan2024-11-141-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 Moldovan2024-11-141-4/+4
| | |
| * | doc/dev/notes: keymgr-certificates: Move subject key specifier to trait.Gabriela Moldovan2024-11-141-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 Moldovan2024-11-141-26/+39
| | | | | | | | | | | | paths.
| * | doc/dev/notes: keymgr-certificates: Mention the uid we settled on in !2577.Gabriela Moldovan2024-11-141-5/+5
| | |
| * | doc/dev/notes: keymgr-certificates: Remove outdated design.Gabriela Moldovan2024-11-141-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 Moldovan2024-11-141-3/+2
| | |
| * | doc/dev/notes: keymgr-certificates: Make the signing_key optionally present ↵Gabriela Moldovan2024-11-141-3/+37
| | | | | | | | | | | | in keystore.
| * | doc/dev/notes: keymgr-certificates: Remove redundant fields from ↵Gabriela Moldovan2024-11-141-5/+0
| | | | | | | | | | | | KeyCertificateSpecifier.
| * | doc/dev/notes: keymgr-certificates: Clarify what KeyType/CertType is about.Gabriela Moldovan2024-11-141-0/+7
| | |
| * | doc/dev/notes: keymgr-certificates: Remove misleading statement.Gabriela Moldovan2024-11-141-1/+0
| | |
| * | doc/dev/notes: keymgr-certificates: Add a revised proposal.Gabriela Moldovan2024-11-141-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 Moldovan2024-11-141-7/+12
| | |
| * | doc/dev/notes: keymgr-certificates: Update key notation for clarity.Gabriela Moldovan2024-11-141-6/+8
| | |
| * | doc/dev/notes: keymgr-certificates: Add note about certificate management.Gabriela Moldovan2024-11-141-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 Jackson2024-11-141-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 Mathewson2024-11-131-5/+11
| |
* | Merge branch 'relay-bin-2' into 'main'David Goulet2024-11-136-6/+216
|\ \ | |/ |/| | | | | arti-relay: add clap for cli See merge request tpo/core/arti!2630
| * arti-relay: add cli testsSteven Engler2024-11-131-0/+35
| |
| * arti-relay: add 'build-info' subcommandSteven Engler2024-11-133-0/+57
| |
| * arti-relay: add clap for cliSteven Engler2024-11-134-2/+113
| | | | | | | | The options are inspired by the arti cli interface.
| * arti-relay: add 'test-all-crates' and 'matrix-check' exceptionsSteven Engler2024-11-132-2/+7
| |
| * arti-relay: add 'default-runtime' featureSteven Engler2024-11-131-2/+4
|/
* Merge branch 'rpc-spec-updates' into 'main'Nick Mathewson2024-11-132-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 Mathewson2024-11-121-1/+1
| |
| * rpc-connect-sketc: Add question on TOML tolerance.Nick Mathewson2024-11-071-0/+6
| |
| * rpc-connect-sketch: more specific abort/decline documentationNick Mathewson2024-11-071-6/+12
| |
| * rpc-connect-sketch: more on philosophy of client error handlingNick Mathewson2024-11-071-2/+34
| |
| * rpc-cookie: shell expansion happens before the addr is canonicalNick Mathewson2024-11-071-0/+2
| |
| * rpc-connect: Clarify server behavior on disabled entries.Nick Mathewson2024-11-071-0/+2
| | | | | | | | | | The behavior in question is to ignore the entries entirely, and not check their files at all.
* | Merge branch 'reconfigure' into 'main'opara2024-11-131-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 Engler2024-11-121-2/+9
|/ / | | | | | | We shouldn't actually reconfigure anything if given `CheckAllOrNothing`.