| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | tor_client: Add some example code for BridgeConfig. | Nick Mathewson | 2023-03-24 | 1 | -0/+49 | |
| | | | | | Closes #791 | |||||
| * | arti-client: Make TransportConfigListBuilder public. | Nick Mathewson | 2023-03-24 | 1 | -1/+8 | |
| | | | | | | | | Without this there's no convenient way to add a transport without messing with serde/toml. Found while working on #791 | |||||
| * | Apply deferred rustfmt churn | Ian Jackson | 2023-03-23 | 1 | -1/+4 | |
| | | ||||||
| * | arti-client: Include redacted HsId in ObtainHsCircuit error | Ian Jackson | 2023-03-23 | 2 | -4/+4 | |
| | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1071/diffs?commit_id=580fe1118dbbbfa086d11b28e3e26d7c91afe21c#fbd7ab2d6d0a5b515697d4f9f99a42ebd508aa4b_153_165 | |||||
| * | arti-client: Fix wording of docs for TorClient::isolation | Ian Jackson | 2023-03-23 | 1 | -1/+1 | |
| | | ||||||
| * | arti-client: Strengthen an HS todo | Nick Mathewson | 2023-03-23 | 1 | -1/+1 | |
| | | ||||||
| * | arti-client: Call tor-hsclient to create HS connections | Ian Jackson | 2023-03-23 | 2 | -4/+27 | |
| | | | | | | Obviously this still doesn't work because the actual impl in hsclient is just a todo!(). | |||||
| * | arti-client: Create an HsClientConnector | Ian Jackson | 2023-03-23 | 2 | -0/+22 | |
| | | ||||||
| * | arti-client: Move client stream creation out of match, in connect | Ian Jackson | 2023-03-23 | 1 | -16/+17 | |
| | | | | | We want this part for HS too. | |||||
| * | arti-client: Break out TorClient::isolation(prefs) | Ian Jackson | 2023-03-23 | 1 | -17/+32 | |
| | | | | | | | | | We're going to want to reuse this. Also: rename StreamPrefs::isolation and update the docs, to better reflect its rather limited functionality. The new TorClient::isolation is the only call site. | |||||
| * | Discuss in comment why we're not using a newtype for the Strings in Host | Ian Jackson | 2023-03-09 | 1 | -0/+8 | |
| | | ||||||
| * | Classify OnionAddressResolveRequest as NotImplemented | Ian Jackson | 2023-03-09 | 1 | -2/+1 | |
| | | | | | | | | | We have a theory about how this ought to work, so it's noit an API usage error - it's an unimplemented feature. As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1060#note_2884757 | |||||
| * | Reject invalid hostnames | Ian Jackson | 2023-03-09 | 1 | -2/+5 | |
| | | | | | | | Reject them both for .onion and for "exit" hostnames This prevents a `Host` from having an invalid hostname in it. | |||||
| * | Fix reference to no-longer-contemplated Host variant | Ian Jackson | 2023-03-09 | 1 | -1/+1 | |
| | | | | | | Reported in https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1060#note_2884755 | |||||
| * | Dummy HsId::from_str on .onion: Return Bug, rather than panicking | Ian Jackson | 2023-03-09 | 1 | -5/+4 | |
| | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1060#note_2884754 | |||||
| * | Improve error from HsId parse of "www.B32.onion" | Ian Jackson | 2023-03-09 | 2 | -0/+3 | |
| | | | | | And add some clarity and todos about address errors. | |||||
| * | Avoid arti_client::ErrorDetail::Address for TorAddrError::InvalidHostname | Ian Jackson | 2023-03-09 | 1 | -1/+19 | |
| | | | | | Provide an explicit From impl isntead. | |||||
| * | arti-client: Parse .onion addresses and dispatch to client.rs | Ian Jackson | 2023-03-09 | 3 | -26/+162 | |
| | | | | | | | | Now, a Host has a separate variant for `.onion` which is always used, even when HS is disabled. Currently it ends in a todo!. | |||||
| * | arti-client: DoThingInstructions: Make error returns consistent | Ian Jackson | 2023-03-09 | 2 | -9/+10 | |
| | | | | | This is more comprehensible, and will be more convenient. | |||||
| * | arti-client: Host: Use Display::fmt rather than write! | Ian Jackson | 2023-03-09 | 1 | -2/+3 | |
| | | | | | | write! is less good because the `{}` overwrites stuff like "did the caller ask for padding". | |||||
| * | Merge branch 'suffix' into 'main' | Nick Mathewson | 2023-03-08 | 1 | -1/+1 | |
| |\ | | | | | | | | | Introduce and use ends_with_ignore_ascii_case See merge request tpo/core/arti!1058 | |||||
| | * | Introduce and use ends_with_ignore_ascii_case | Ian Jackson | 2023-03-08 | 1 | -1/+1 | |
| | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1056#note_2884428 | |||||
| * | | Merge branch 'host2' into 'main' | Ian Jackson | 2023-03-08 | 2 | -47/+140 | |
| |\ \ | |/ |/| | | | | | arti-client: Do not send resolve requests for IP addrs to exits See merge request tpo/core/arti!1057 | |||||
| | * | arti-client: Host: Be more explicit about what is allowed in Hostname | Ian Jackson | 2023-03-08 | 1 | -1/+11 | |
| | | | ||||||
| | * | Rustfmt | Ian Jackson | 2023-03-08 | 2 | -4/+16 | |
| | | | ||||||
| | * | Abolish Host::into_string_and_port | Ian Jackson | 2023-03-08 | 1 | -23/+0 | |
| | | | | | | | | | | | | | | | | | | | When I was trying to add HS support to these layers, I found I could add a new variant to the `Host` enum but everything would still compile even though I hadn't written the necessary implementation! This method is a liability: when using it, one inevitably writes such latent bugs. | |||||
| | * | arti-client: Do not send resolve requests for IP addrs to exits | Ian Jackson | 2023-03-08 | 2 | -13/+58 | |
| | | | | | | | | | | | | | | | | | Doing so doesn't seem like a good idea. It might even be some kind of leak? Found because I added a variant to `address::Host` for hidden services, and noticed that the resolve code still compiled. | |||||
| | * | arti-client addr handling: Introduce StreamInstructions | Ian Jackson | 2023-03-08 | 2 | -22/+71 | |
| | | | ||||||
| * | | arti-client: Reject .onion, when we should, without allocating | Ian Jackson | 2023-03-08 | 1 | -1/+2 | |
| |/ | | | | Use new strip_suffix_ignore_ascii_case | |||||
| * | Rename onion-* cargo features to hs-* in tor-* crates | Ian Jackson | 2023-02-28 | 2 | -2/+2 | |
| | | | | | Fixes #756 | |||||
| * | Merge branch 'shadykaty-error-hint-v2' into 'main' | eta | 2023-02-07 | 2 | -1/+140 | |
| |\ | | | | | | | | | | | | | Create and use an ErrorHint type to report how to fix a problem. Closes #579 and #578 See merge request tpo/core/arti!994 | |||||
| | * | ErrorHint: Add a note about lowering the permissions hint | Nick Mathewson | 2023-01-27 | 1 | -0/+4 | |
| | | | ||||||
| | * | ErrorHint: use anonymize_home | Nick Mathewson | 2023-01-27 | 1 | -2/+4 | |
| | | | ||||||
| | * | ErrorHint: Implement tryfrom_torpersist | Nick Mathewson | 2023-01-27 | 1 | -2/+6 | |
| | | | ||||||
| | * | ErrorHint: refactor API (part 2) | Nick Mathewson | 2023-01-27 | 1 | -75/+60 | |
| | | | | | | | | | | | | | | | Change ErrorHint so that, internally, it just holds an enum with a lightweight reference to whatever parts of the error it needs to generate a hint. Then we can move the formatting logic into a Display function for ErrorHint, and do away with ErrorDetail entirely. | |||||
| | * | ErrorHint: refactor API (part 1) | Nick Mathewson | 2023-01-27 | 1 | -13/+19 | |
| | | | | | | | | | | | | | | | Move the "hint" function into Error, and use Option rather than Result. (I'm using Option here because it's not really an error case not to have a hint; we just either have a hint, or we don't.) | |||||
| | * | add opaque ErrorHint API, impl ErrorHint from BadPermissions | Shady Katy | 2023-01-27 | 2 | -1/+139 | |
| | | | ||||||
| * | | Use ErrorReport for errors in error! in arti-client | Ian Jackson | 2023-01-30 | 2 | -3/+4 | |
| |/ | ||||||
| * | Allow clippy::unchecked_duration_subtraction in tests | Nick Mathewson | 2023-01-27 | 4 | -0/+4 | |
| | | | | | | This panics on error, and we're fine with a panic on misbehavior in tests. | |||||
| * | Disable clippy::unlinlined-format-args | Nick Mathewson | 2023-01-27 | 1 | -0/+1 | |
| | | | | | | | | | This warning kind of snuck up on us! (See #748) For now, let's disable it. (I've cleaned it up in a couple of examples, since those are meant to be more idiomatic and user-facing.) Closes #748. | |||||
| * | made TorClient::set_stream_prefs public | Michael van Straten | 2023-01-18 | 1 | -4/+1 | |
| | | ||||||
| * | Make a top-level API for connecting to onion services. | Nick Mathewson | 2023-01-12 | 2 | -0/+16 | |
| | | | | | Fortunately, it adds very little to the arti-client API surface. | |||||
| * | test lint blocks: Add many many automatically | Ian Jackson | 2022-12-12 | 4 | -0/+33 | |
| | | | | | | This is precisely the result of running the rune in maint/adhoc-add-lint-blocks. | |||||
| * | arti-client: spelling fix. | Nick Mathewson | 2022-12-06 | 1 | -1/+1 | |
| | | ||||||
| * | arti: BridgesConfig is evidently not too high up | Ian Jackson | 2022-11-30 | 1 | -8/+0 | |
| | | | | | | Since apparently it was possible to write all the other code without moving it. | |||||
| * | arti: pt mgr spawns the task itself in new | Ian Jackson | 2022-11-30 | 1 | -3/+0 | |
| | | ||||||
| * | arti-client: Try to take ownership of state earlier. | Nick Mathewson | 2022-11-29 | 1 | -0/+4 | |
| | | | | | | | | | There's no harm in doing this, since we'll try to take ownership of the state soon as soon as we use it: We just want to try early, so that we'll get it before we decide that we're using bridges. Fixes bug #669; bug introduced by !889. | |||||
| * | arti, arti-client: Conditionalise various things on pt-client | Ian Jackson | 2022-11-29 | 1 | -3/+4 | |
| | | ||||||
| * | Spelling fix in arti_client::err | eta | 2022-11-29 | 1 | -1/+1 | |
| | | ||||||
| * | Temporarily make ptmgr required for arti-client :/ | Nick Mathewson | 2022-11-28 | 1 | -0/+3 | |
| | | ||||||
