| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| |\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | | |
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
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
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.
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This is mostly a matter of encoding parameters in the format used
for socks authentication.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This code is _not_ conditional, since we want to support running
with a proxy even if we don't support pluggable transports.
We may eventually want to refactor this into a new create.
|
| | | | | | |
|
| | | | | | |
|
| |\ \ \ \ \
| |_|/ / /
|/| | | |
| | | | |
| | | | | |
Use BridgeConfig to identify bridges in two places
See merge request tpo/core/arti!781
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This lets us write functions which can either take an existing
owned OwnedChanTarget, or copy out of some other kind of ChanTarget
passed by reference.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The target is identified by Into<OwnedChanTarget>.
We introduce corresponding TargetCircUsage and SupportedCircUsage.
|
| | | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | | |
Now keyed by Arc<BridgeConfig>, and the values can be errors.
Currently there is no implementation so there can't be any errors,
but the error enum will become nonempty.
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
impl ChanTarget for BridgeConfig
See merge request tpo/core/arti!783
|
| | | | | |
|
| | | | | |
|
| |/ / /
| | |
| | |
| | | |
Eg, a PT bridge may not have an address.
|
| |/ /
| |
| | |
The "shadow-small" tag no longer exists, and all "tpa" x86-64 runners can now run Shadow. https://gitlab.torproject.org/tpo/tpa/team/-/issues/40833#note_2835576
|
| |\ \
| | |
| | |
| | |
| | | |
shadow ci: Set stop time to 30m instead of 27m
See merge request tpo/core/arti!772
|
| | |/
| |
| |
| |
| | |
This was a temporary stop-gap to work around a busy loop that seemed to
happen before. I am no longer able to reproduce any such problem.
|
| |\ \
| | |
| | |
| | |
| | | |
fix a couple of spelling errors in guardmgr
See merge request tpo/core/arti!778
|
| |/ / |
|
| |\ \
| | |
| | |
| | |
| | | |
Several spelling fixes.
See merge request tpo/core/arti!777
|
| | | | |
|
| |/ / |
|
| |\ \
| | |
| | |
| | |
| | | |
ChanMgr: Revise code to tolerate multiple identities
See merge request tpo/core/arti!773
|
| | | |
| | |
| | |
| | |
| | | |
This can happen in weird corner cases, so it's probably best to
report it rather than having an "internal error."
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Now, instead of duplicate checks in various cases, we simply go
through the loop one last time.
This allows us to simplify some of our other logic around here.
|
| | | |
| | |
| | |
| | |
| | | |
This function had grown huge and hard to reason about. Before I
make it even worse, let's split it up.
|
| | | | |
|