| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | |
| | | |
| | | |
| | | | |
The singleton variation here is almost never what we want.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Now it contains either an `OwnedChanTarget` or an `OwnedCircTarget`,
which will let `GuardMgr` return bridges that can be used to make
circuits.
As part of this change, it was necessary to revise some
address-modification functions that applied to filters and
`OwnedChanTarget`. Now they do the smart thing, and remove only the
address that are in the `ChanMethod`. This means that the addresses
from HasAddrs are still accurate about which addresses the relay
"has".
|
| | | | |
| | | |
| | | |
| | | | |
This resolves an old TODO, and will simplify our work a little.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The most important part of this commit is to make sure that each
`FirstHopId` includes the `GuardSetSelector` from which the guard
was selected. Doing this lets us be certain that when we report
that a guard has succeeded or failed, we're reporting it in the
right context.
Additionally, this commit uses strum to make an iterator over the
samples, so that we can make sure that our "for each sample" code is
robust against future changes, and we don't miss the bridge sample.
|
| |\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | | |
Add two ways to change entries in an n_key_set
See merge request tpo/core/arti!797
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | |/ /
| | |
| | |
| | |
| | |
| | | |
The keys can change: if they do, then the entry is removed from its
previous position and inserted at the new one, possibly displacing
others.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
tor-cell: Fix typos in msg.rs
See merge request tpo/core/arti!788
|
| | | |/
| |/| |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
tor-cell: Remove redundant match clauses
See merge request tpo/core/arti!792
|
| | |/ /
| | |
| | |
| | |
| | | |
This commit removes two redundant match clauses inside the
take_one_netinfo_addr function found inside msg.rs.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
tor-cell: Make historical comments more concrete
See merge request tpo/core/arti!787
|
| | |/ /
| | |
| | |
| | |
| | | |
This commit extends comments that make references to historical protocol
versions of Tor, by adding the concrete protocol version numbers.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
tor-circmgr make get_or_launch_dir_specific depend on specific-relay feature
See merge request tpo/core/arti!795
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
DirSpecificTarget in the usage was renamed apropos an MR comment but
this function name was overlooked.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
If pt-client is disabled, there are in fact no tests, so the module is
empty. But we would always want `use super::*` here.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This doesn't make sense now we've released Arti 1.0. The version
number is enough I think.
|
| | | |/
| |/| |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
cleanup target dir on job completion
See merge request tpo/core/arti!786
|
| | | | | |
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
tor-cell: Avoid redundant pointer copy
See merge request tpo/core/arti!791
|
| | | |/
| |/|
| | |
| | |
| | |
| | | |
This commit changes an iteration by copying a u16 (which is 2 bytes)
instead of a pointer address, which is most likely 8 bytes on most
machines.
|
| |\ \ \
| |_|/
|/| |
| | |
| | | |
Make error from bridge descriptor provider be a trait object
See merge request tpo/core/arti!796
|
| | | | |
|
| | | | |
|
| |/ /
| |
| |
| |
| |
| |
| |
| | |
We could in the future provide a version of abs_retry_time which took
an &dyn closure if that turns out to be wanted.
I think this isn't a semver break because trait implementors are
allowed to relax bounds.
|
| |\ \
| | |
| | |
| | |
| | | |
Oops: Mark some imports as pt-client only.
See merge request tpo/core/arti!794
|
| | | |
| | |
| | |
| | | |
This causes CI problems, and is entirely my fault.
|
| |\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | | |
add checks for many feature combinations
Closes #303
See merge request tpo/core/arti!775
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | | |
and fix issue compiling tor-rtcompat with on ssl runtime
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
tor-cell: Avoid redundant allocation
See merge request tpo/core/arti!790
|
| | | |/
| |/|
| | |
| | |
| | |
| | | |
This commit changes the way how a vector with a known sized gets
allocated, by using Vec::with_capacity() instead of Vec::new().
It will eventually avoid an allocation of more memory than required.
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
tor-cell: Rename fixed_len to fixed_len_handshake
See merge request tpo/core/arti!789
|
| |/ /
| |
| |
| |
| |
| | |
This commit renames the fixed_len! macro to fixed_len_handshake!, in
order to indicate, that this macro is only suited for cells with
commands related to handshaking.
|
| |\ \
| | |
| | |
| | |
| | | |
Make DirResponse API not an error handling footgun
See merge request tpo/core/arti!782
|
| | | | |
|
| | | |
| | |
| | |
| | | |
I want this for testing. It seems friendly to provide it, though.
|
| | | | |
|
| | | |
| | |
| | |
| | | |
Split off to assist review.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
We're going to have functions on Response that fail by returning only
one of these.
Sadly this diff is quite noisy.
|
| | | |
| | |
| | |
| | |
| | | |
These don't check errors and are therefore quite hazardous.
I'm going to introduce a more cooked version in a moment.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
serde support for PtTarget
See merge request tpo/core/arti!780
|
| | | | | |
|