summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | connect points: Validate more correctness propertiesNick Mathewson2024-11-191-4/+23
| | | | | | | | | | | | | | | | | | | | In particular, we only allow binding/connecting to localhost, and we only support None auth for Unix connections.
| * | | connect points: Remember original string addressesNick Mathewson2024-11-193-6/+61
| | | | | | | | | | | | | | | | The cookie authentication protocol will need these.
| * | | rpc-connect: Types for cookie authentication.Nick Mathewson2024-11-194-1/+193
| | | | | | | | | | | | | | | | (Cookie authentication is described in rpc-cookie-sketch.md)
| * | | rpc-connect: Move tempdir() helper to a new moduleNick Mathewson2024-11-193-17/+36
| | | |
| * | | RPC connect points: Facility to load from files.Nick Mathewson2024-11-194-1/+427
| | | |
| * | | connect point: implement FromStr.Nick Mathewson2024-11-191-17/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While doing so, tidy up the errors that we detect when parsing a connect point, so that we can differentiate ones that indicate an unsupported connect-point type. This means that fewer types now implement Deserialize; that's probably a good thing.
| * | | tor-connect-point: new crate to manage RPC connect pointsNick Mathewson2024-11-196-27/+558
| | | | | | | | | | | | | | | | See doc/dev/rpc-book/src/rpc-connect-sketch.md for details.
| * | | CfgAddr: Function to check whether substitutions applyNick Mathewson2024-11-191-0/+10
| | | | | | | | | | | | | | | | We'll need this for implementing the connect point logic.
| * | | tor-config-path: Note an unused error variantNick Mathewson2024-11-191-0/+2
|/ / /
* | | Merge branch 'config-paths' into 'main'opara2024-11-1817-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 Engler2024-11-181-6/+12
| | | |
| * | | tor-config-path: add more documentation to `CfgPathResolver`Steven Engler2024-11-181-0/+9
| | | |
| * | | tor-config-path: remove `Option` from resolver valueSteven Engler2024-11-182-22/+28
| | | |
| * | | arti-client: add path resolver to `TorClient` and `TorClientConfig`Steven Engler2024-11-184-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 testSteven Engler2024-11-181-10/+35
| | | |
| * | | arti-client: moved two tests from tor-config-pathSteven Engler2024-11-182-23/+20
| | | | | | | | | | | | | | | | Also updated to use the `PATH_RESOLVER` resolver.
| * | | update `CfgPath::path` to use a `CfgPathResolver`Steven Engler2024-11-1813-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 Engler2024-11-183-3/+19
| | | |
| * | | tor-hsservice: pass through the `CfgPathResolver`Steven Engler2024-11-186-15/+55
| | | |
| * | | tor-config-path: begin using the `CfgPathResolver`Steven Engler2024-11-181-20/+28
| | | |
| * | | tor-config-path: add unused `CfgPathResolver`Steven Engler2024-11-181-1/+60
| | | |
| * | | tor-config-path: moved 'impl Display for CfgPath' blockSteven Engler2024-11-181-9/+9
| | | |
| * | | tor-config-path: moved two testsSteven Engler2024-11-181-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 Jackson2024-11-182-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 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.
generated by cgit v1.2.3 (git 2.46.0) at 2026-08-27 21:24:19 +0000