| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | |
| | | |
| | | |
| | | |
| | | | |
This (briefly) regresses the documentation, but it's going to appear
again when we derive the methods.
|
| |/ / / |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
We decided to move away from the `Error` + `ErrorDetail`
all-encompassing enum design, and will use `anyhow::Error` in more
places. When we need to handle specific errors, we will create smaller
more-specific error types in those cases.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Use random_hostname() to pick an (E)SNI
Closes #2210
See merge request tpo/core/arti!3372
|
| | |/ /
| | |
| | |
| | |
| | |
| | | |
Closes #2210.
Edited-by: Nick Mathewson <[email protected]>
|
| | | | |
|
| |/ /
| |
| |
| | |
Closes #2067.
|
| |\ \
| | |
| | |
| | |
| | | |
Add more detail on retry times to GuardMgr logging
See merge request tpo/core/arti!3153
|
| | | |
| | |
| | |
| | |
| | | |
While this is fairly nested, it's short, and most of the complexity is
just in the logging code.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
The "Next retry time unknown" message can't be hit right now, but could
be if the code changes in the future.
|
| | | |
| | |
| | |
| | |
| | | |
I don't love shoving such a complicated expression in thiserror, but it
seems nicer than writing the implementation by hand...
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
arti: Ignore reload_cfg::test::watch_multiple.
See merge request tpo/core/arti!3364
|
| | |/ /
| | |
| | |
| | | |
This is flaky, ignoring until #1607 is fixed.
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
proto: Split out IncomingDataCmdChecker as a separate type
See merge request tpo/core/arti!3348
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
The previous "incoming" terminology was rather ambiguous.
Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3348#note_3275337
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
This will be used by relays too (for validating incoming messages on
streams).
This is just code motion, so it's best reviewed with `--color-moved`.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The incoming one will be used for the exit relay implementation too.
Also, with this change, receiving `CONNECTED` on an incoming stream will
result in a clearer error message. Previously, the check against
receiving `CONNECTED` on an incoming stream was bundled with the
double-CONNECTED check for client data streams, so in the incoming
stream case, the error message was misleading ("Received CONNECTED twice
on a stream.").
|
| | | |
| | |
| | |
| | |
| | | |
`Arc` is already in scope, and not fully-qualifying it makes it more
readable.
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
This makes it easier to see which parts are implementation-agnostic
(i.e. do not import from crate::client).
|
| | | |
| | |
| | |
| | |
| | | |
The CmdChecker will be used by relays too, so I am moving it to the
shared `stream` module.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
arti config: Fix flaky watch_single_file test.
Closes #1607
See merge request tpo/core/arti!2503
|
| | | | |
| | | |
| | | |
| | | | |
Thanks to gabi for the suggestion.
|
| | | | |
| | | |
| | | |
| | | | |
Fixes: #1607
|
| | | | | |
|
| | |/ /
|/| | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
proto: Fix typo in OutboundRelayLayer docs.
See merge request tpo/core/arti!3346
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
`OutboundRelayLayer::decrypt_outbound()` is for decrypting cells moving
*away* from the client (in the "forward direction").
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
tor-netdoc: Be able to parse routerstatus entries in votes
See merge request tpo/core/arti!3351
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Add links to torspec, and other cross-references.
Change heading fof `IgnoredPublicationTimeSp` to `publication` to
match spec and distinguish it from the `published` item in netstatus
docs.
Align several descriptions with the wording from torspec.
Remove a restatement of a fact from the spec, on the `valid_until`
field in `Lifetime`.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
I have eyeballed the output and it seems to look right.
Eventually we'll have encoing code and then we'll be able to check
that things round-trip.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This is a test of real document structs now, albeit that it still uses
one of the types from `poc`.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This now exercises all the new parsing code.
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
In md consensuses the referenced document digest is in the `m` field.
This seems to be to make it similar to votes. But really the function
of the md desc hash in md rs entries is much more parallel to the
plain desc hash in plain rs entries. The way the spec has done it
means the `r` item has a needlessly different syntax.
|
| | | | | | |
|