| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | |
| | | |
| | | |
| | | | |
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.
|
| | | |/
| |/| |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
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.
|
| | | | |
|
| | | | |
|
| | |/
|/|
| |
| |
| |
| |
| |
| | |
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.
|
| |/ /
| |
| |
| |
| |
| | |
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
|
| | | | |
| | | |
| | | |
| | | | |
This will let us remember bridges that use pluggable transports.
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The `PtTarget` type and its contents (`TransportName`,
`PtTargetAddr`, `PtSettings`) are now unconditionally compiled and
exposed. This will allow us to serialize and deserialize them in
our guard-state files even when we have been built without explicit
PT support.
The `pt-client` feature controls whether `TransportName` is a
variant of `TransportId`, and whether `PtTarget` is a variant of
`ChanMethod`: this in turn means that we'll still have simpler
binary code and smaller structures when we're building without PT
support (which is what we wanted when we initially made these types
conditional).
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Implement a TransportHelper to connect via a SOCKS-based pluggable transport
See merge request tpo/core/arti!776
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This is a compromise between C tor (never escapes = signs) and the
spec (calls for escaping all = signs). In C tor there is no way to
actually construct a key with an = sign in it, so this difference
should be undetectable.
|
| | | | | | |
|
| | | | | | |
|