| Commit message (Collapse) | Author | Age | Files | Lines |
| | |
|
| |
|
| |
Co-authored-by: gabi-250 <[email protected]>
|
| |
|
|
|
|
| |
Otherwise we either lose the property that you can call
launch_onion_service before bootstrapping has begun,
or we have to make launch_onion_service async.
|
| |
|
|
|
| |
(This is a separate commit in order make the code movement in the
previous commit clearer.)
|
| |
|
|
|
| |
This will ensure that the TorClient takes its file locks immediately
on construction.
|
| | |
|
| | |
|
| |
|
|
|
|
| |
This will let us be more confident in the inertness of a
not-bootstrapping TorClient, _and_ make it easier to reconfigure
things before the client is started.
|
| |
|
|
|
| |
Now that we have a type called "inner", and will soon have a
function to create it, we really don't want to retain this name.
|
| |
|
|
|
| |
We want this look to cover the whole function, to avoid weird
duplicate behavior while creating our bridge desc manager.
|
| |
|
|
|
|
| |
We'll use this to distinguish "not running" from "running",
in order to make it easier to be sure that non-bootstrapping clients
will definitely not try to connect to the network.
|
| | |
|
| | |
|
| |
|
|
|
|
|
| |
Previously the [channel] outbound_proxy setting was applied to Arti's
own direct connections but was not forwarded to managed PT processes.
https://spec.torproject.org/pt-spec/configuration-environment.html
|
| |
|
|
|
|
|
|
| |
Discussion on #1960 suggests that this option is not a good idea:
it encourages developers to work around deliberate signals that the
software they're shipping won't work on the network.
Closes #1960.
|
| | |
|
| | |
|
| |
|
|
| |
These were only Arc<> so that TorClient could implement Clone.
|
| |
|
|
|
|
|
|
|
| |
The `set_stream_prefs` method was the only one that took a TorClient
as `&mut self`. But we only expose Arc<TorClient<R>>, that isn't a
reasonable API.
The `clone_with_prefs` method has been renamed, since it no longer
"clones" anything meaningfully.
|
| | |
|
| |
|
|
|
|
| |
This is part of rationalizing the structure of TorClient so we can
refactor startup logic, and so that RPC code can reason about object
identity. See #2469.
|
| | |
|
| | |
|
| |
|
|
| |
As per the release checklist.
|
| | |
|
| |
|
|
|
|
|
|
| |
The replacement (TestingStateMgr) is not correct, but it will
compile until we have a supported state manager.
(This made cargo-sort have me move a section around. Not sure why it
cares.)
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
(There are a few places where we still do things in nonstandard
places to avoid API breakage.)
|
| |\
| |
| |
| |
| |
| |
| | |
Use ErrorSources where appropriate to walk an error stack.
Closes #2338
See merge request tpo/core/arti!3685
|
| | |
| |
| |
| |
| | |
Previously, due to #2338, we would miss any error nested directly
inside an io::Error.
|
| |/
|
|
|
|
|
|
|
| |
`clippy::collapsible_if` started triggering after bumping the MSRV to
1.88.
Since this triggers from a lot of places, and since there even are a
couple of instances where we explicitly allow `clippy::collapsible_ifs`,
I've opened #2342 for deciding what to do about it.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This requires the `TlsKeyAndCert` so be passed on the TLS acceptor
settings. We assume that `RelayIdentities` has this information.
The ChanBuilder::new() was getting a bit too convoluted and feature
gated to instead we introduce new_client() and new_relay() and remove
the need for `with_identities()`.
Because of this, the ChanMgr::new() now returns a `Result<>`.
Related to #1597
Signed-off-by: David Goulet <[email protected]>
|
| | |
|
| |
|
|
| |
This adds the lint to all our crates.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This struct is used to pass configuration parameters to the ChanMgr when
building it.
At the moment, it holds the ChannelConfig and RelayIdentities (feature
gated) which will be used in subsequent commits.
Note that relays do require RelayIdentities to build channels.
Signed-off-by: David Goulet <[email protected]>
|
| |
|
|
|
|
|
| |
We'll rely on a RelayIdentities to pass in the right keys to the ChanMgr
instead of the entire KeyMgr.
Signed-off-by: David Goulet <[email protected]>
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
| |
Fixes part of #2193.
(Edits from nickm: I selected the cases here that I could verify
were correct from immediate context.)
Edited-by: Nick Mathewson <[email protected]>
|
| |\
| |
| |
| |
| | |
Add `KeyMgr` accessor to `TorClient`
See merge request tpo/core/arti!3442
|
| | | |
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | | |
arti-client: Fix typo
See merge request tpo/core/arti!3504
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Now that `vanguard_config()` is no longer conditional, we can handle
`vanguard_config` just like other accessors. This is a breaking change,
but I think we may as well make this change while we've already made
another breaking change in the previous commit.
|
| | | | |
|
| |/ /
| |
| |
| |
| | |
These are all aimed at figuring out in more detail what's going on
in #2079 and related issues.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
arti/arti-client/tor-hsservice: Support disabling onion services in the config
Closes #2133
See merge request tpo/core/arti!3253
|