| Commit message (Collapse) | Author | Age | Files | Lines |
| | |
|
| |\
| |
| |
| |
| | |
tor-netdoc: Handle anomalous numbers of introduction points
See merge request tpo/core/arti!1332
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| |
| |
| | |
None of the existing NetdocErrorKinds seemed right.
|
| |/
|
|
|
| |
I was going to add a comment about "doing this if we need the voting
period anywhere else" but it turns out that we also use it in dirmgr.
|
| |
|
|
|
| |
Fixes a warning with
cargo clippy --locked --offline --workspace --all-targets
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
=========================
Notes from nickm:
(This differs from pinkforest's original MR: It removes the
Cargo.lock changes and the version bump on tor-llcrypto.)
Minimal Cargo.lock changes from downgrade.
(These are exactly those changes generated by running "build" and
"test".)
There are several reasons to do this:
* It's best to bump all of our dalek dependencies at once to rc.3
or later, rather than the piecemeal approach we've been stuck
with so far.
* We don't want to do this bump right now, since there are some
tricky questions about clamping we need to figure out (see
#808), and we need to make sure we get them right, and we're in
a distracted this week.
* We _do_ need to move away from 2.0.0-rc.2 right now, since
it was causing a failure in `cargo install arti`, and then it
got yanked.
Thanks to pinkforest for helping us out here and explaining all of
this!
Fixes #926.
Commit-edited-by: Nick Mathewson <[email protected]>
|
| |\
| |
| |
| |
| | |
netdoc: Resolve nearly all remaining TODO HS items.
See merge request tpo/core/arti!1304
|
| | |
| |
| |
| | |
Although this Errorkind is not perfect, it is good enough.
|
| | | |
|
| | |
| |
| |
| |
| |
| | |
Actually, never mind about adding an accessor here: this value was
set incorrectly and didn't match its documentation. As such it's
basically useless, and we might as well throw it out.
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This commit removes some actual dead code and additionally adds some
minimal accessors to HsDesc to expose some of its properties. (I'm
trying to keep these minimal since it's not yet clear whether we
want to expose more detail here.)
Here we also make StoredHsDescMeta a conditional type that's only
present when the new "hs-dir" feature is enabled. Neither relays
nor clients need this: Only HsDirs will need it, when we finally
implement relays.
|
| | |
| |
| |
| | |
(We explicitly do not care if Arti can be distinguished from C tor.)
|
| | |
| |
| |
| | |
The tests called for here already existed.
|
| | |
| |
| |
| |
| | |
This resolves some TODO HS items now that we are confident we want
to have these actual examples in our tests.
|
| |/ |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
For higher-level HsDesc-related functions, this type indicates where
exactly an error occurred. It lets us distinguish decryption errors
from parsing errors, and attribute responsibility to the hsdir, the
onion service, or the user's lack of encryption.
This lets us remove some just-introduced complexity in
tor-hsclient.
|
| |\
| |
| |
| |
| |
| |
| | |
Generate correct-ish socks5 errors for onion service errors.
Closes #736
See merge request tpo/core/arti!1279
|
| | |
| |
| |
| | |
Part of #736
|
| |\ \
| | |
| | |
| | |
| | | |
lints: Promote clippy::print_stderr and clippy::print_stdout
See merge request tpo/core/arti!1271
|
| | |/ |
|
| | | |
|
| |/
|
|
| |
Closes #909.
|
| |
|
|
|
|
| |
The last piece here is removing some needless keygen from the
tests. This causes the test output (using the deterministic PRNG)
to change.
|
| | |
|
| |
|
|
|
| |
The value here is 32 bytes; I'm adding it to to the specification
as torspec!141
|
| | |
|
| | |
|
| |
|
|
|
| |
(I doubt that there is a viable side channel here, but still we
might as well get in the habit of doing these things properly.)
|
| | |
|
| |
|
|
| |
I'm using HSS for hs-service features.
|
| |
|
|
|
|
|
|
|
|
|
| |
This could never be a full rustdoc test, since rustdoc never
runs tests for private items. Even if it were a rustdoc test,
it uses a bunch of types that aren't exposed in the right places,
and it invokes nonexistent functions, and it assumes a `self`
that isn't there.
In lieu of writing a new untested test, I've added a note to refer
the developer to where they can find working example code.
|
| | |
|
| | |
|
| |\
| |
| |
| |
| | |
Construct a CircTarget from an IntroPointDesc.
See merge request tpo/core/arti!1221
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The consensus includes a listing for clients and for relays,
saying which protocol versions are _required_ for participation on
the network, and which versions are _recommended_. We have been
parsing this, but not yet exposing it.
This commit adds accessors to expose it, since we'll need that in
order to create CircTargets for introduction points and rendezvous
points.
|
| |\|
| |
| |
| |
| | |
hsdesc docs: Provide some accessors
See merge request tpo/core/arti!1220
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| | |
These are (I hope) the only key matching the description. If they're
not, the description needs to be expanded.
|
| |/ |
|
| |
|
|
|
|
| |
Fortunately, these are all in experimental code.
Closes #798
|
| | |
|
| |
|
|
| |
The `new` function is deprecated in x25519-dalek 2.0.0-rc.2
|