| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | |/ / / /
| |/| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
"Backward" because this reactor will deal with relaying cells in the
backward direction (from exit to client). In addition, this reactor will
deal with stream handling and control/command messages.
We will soon have another, "forward", reactor, relaying cells in the
forward direction.
|
| |\ \ \ \ \ \
| |/ / / / /
|/| | | | |
| | | | | |
| | | | | | |
tor-netdoc: impl NetdocParseableFields for Lifetime
See merge request tpo/core/arti!3379
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
These fields aren't public so this doesn't have semver impacts.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
In a moment, we're going to want to transform the field arguments.
That would leave us open to bugs like this:
valid_after: valid_after.into(),
fresh_until: fresh_until.into(),
valid_until: valid_after.into(),
We'll sort out the indentation later in this branch.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This (briefly) regresses the documentation, but it's going to appear
again when we derive the methods.
|
| | |/ / / / |
|
| |\ \ \ \ \
| |/ / / /
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | | |
CI: Specify versions for shellcheck and editorconfig-checker.
Closes #2219
See merge request tpo/core/arti!3366
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Fixes: #2219
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
arti-relay: Refactor error handling
See merge request tpo/core/arti!3374
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
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.
|
| |\ \ \ \ \ \
| |/ / / / /
|/| | | | |
| | | | | |
| | | | | | |
doc: Add note about relays enforcing protocol recommendations
See merge request tpo/core/arti!3373
|
| | | |_|/ /
| |/| | | |
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
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]>
|
| |\ \ \ \ \
| |/ / / /
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Refactor: turn circ_extensions_from_settings into a method
Closes #2067
See merge request tpo/core/arti!3344
|
| | | | | | |
|
| |/ / / /
| | | |
| | | |
| | | | |
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.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
ci: Parallelize minimal-features-test
Closes #2207
See merge request tpo/core/arti!3338
|
| | |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
3 seems like a good number to start with, we can tune it later if
needed.
Fixes: #2207
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
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
|
| |\ \ \ \ \
| |_|/ / /
|/| | | |
| | | | |
| | | | | |
Fix a couple of clippy warnings from nightly
See merge request tpo/core/arti!3361
|
| | | | | | |
|
| | | | | | |
|
| |\ \ \ \ \
| |/ / / /
|/| | | |
| | | | |
| | | | | |
maint/reproducible_build: Use cp -R to placate shellcheck
See merge request tpo/core/arti!3362
|
| |/ / / /
| | | |
| | | |
| | | |
| | | |
| | | | |
I'm not sure I agree with this complaint - it seems like foolish
standards-wonkery - but GNU cp supports -R too and this change will
fix CI on main.
|