| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | |
|
| | | |
| | |
| | |
| | | |
This will be needed for ntor handshakes.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
This also updates the key rotation task to call the setter whenever the
ntor keys get updated.
|
| | | |
| | |
| | |
| | | |
This will need to be updated each time the ntor keys change.
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This trickles down to the tor-proto channel handshake code. But, the
real need is in the channel builder in order to validate the outbound
channel target.
Fixes #2440
Signed-off-by: David Goulet <[email protected]>
|
| |\ \
| | |
| | |
| | |
| | | |
tor-proto: Move CREATE_FAST handling to a helper
See merge request tpo/core/arti!3869
|
| | | |
| | |
| | |
| | |
| | | |
Clippy has started warning about this since we moved the CREATE_FAST
handling to a helper, so this resolves that.
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | | |
Fix formatting from previous code movement.
|
| | | |
| | |
| | |
| | |
| | | |
This moves the code, changes the indentation, and wraps the result in an
`Ok()`.
|
| | | | |
|
| | | |
| | |
| | |
| | | |
This had already been resolved.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
From opara's comment:
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3844#note_3388789
Keep the low level AuthLogDigest type alias and return it. The callsite
is the one deciding if the returned digest is a Clog or a Slog.
Related to #2441
Signed-off-by: David Goulet <[email protected]>
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| | |
Introduce those types in order to avoid mixing them up as the previous
AuthLogDigest was just a type alias over [u8; 32]
Fixes #2441
Signed-off-by: David Goulet <[email protected]>
|
| |\ \
| | |
| | |
| | |
| | | |
Add support for handling CREATE_FAST cells and launching a circuit reactor
See merge request tpo/core/arti!3846
|
| | | | |
|
| | | |
| | |
| | |
| | | |
This reverts commit 9c38daf2d3548feca2ff555f5bd52165add0d20c.
|
| | | |
| | |
| | |
| | | |
This is needed for the revert in the following commit.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Instead of giving the `CreateRequestHandler` to the channel after it's
constructed, we integrate it into the handshake so that we can give it
to the channel constructor.
The `ChannelType` is no longer part of the `Channel`.
Some of the tests could be cleaned up slightly now that the channel
doesn't need the `ChannelType`, but I don't want to conflict with !3853.
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
We can't pass the request handler during the `Channel` constructor since
it would require conditionally compiled function arguments, which aren't
nice.
|
| | | |
| | |
| | |
| | | |
for handling CREATE* messages on channels.
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
This better fits with existing code that uses `restricted_msg!` and is a
bit more flexible.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
I think it was written like this because earlier versions of rust didn't
allow references to temporaries like `&mut rand::rng()` and needed them
to be bound to a variable. But this isn't necessary anymore and so the
comment doesn't make sense.
|
| | | |
| | |
| | |
| | |
| | | |
When used for relay channels, the channel reactor will soon need to
spawn relay circuit reactors.
|
| | | | |
|
| | | |
| | |
| | |
| | | |
And rename `CircMap::add_ent()` to `add_origin_ent()`.
|
| | | | |
|
| | | | |
|
| |\ \ \
| |_|/
|/| |
| | |
| | |
| | |
| | | |
memquota: Add new add_parent() API
Closes #2427
See merge request tpo/core/arti!3829
|
| | | |
| | |
| | |
| | | |
Closes #2427
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
tor-proto: Change an error from `CircuitClosed` to `NotConnected`
Closes #2421
See merge request tpo/core/arti!3825
|
| | |/ /
| | |
| | |
| | |
| | | |
Returning `CircuitClosed` isn't right here since the circuit may not
have closed.
|
| | | |
| | |
| | |
| | | |
This appears often in the arti logs, but is a normal thing to happen.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
tor-proto: Allow sending DATA cells on closed streams
Closes #2434
See merge request tpo/core/arti!3824
|
| | |/ /
| | |
| | |
| | |
| | | |
Previously we would close the circuit, which isn't great because there
can be other streams in use on the circuit.
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Prefer runtime.now/wallclock to SystemTime/Instant::now, when available.
Closes #2428
See merge request tpo/core/arti!3845
|
| | | |/
| |/|
| | |
| | |
| | |
| | |
| | | |
Previously we'd get the time by calling Instant::now, but that would
lead to a mismatch with the runtime if we were mocking.
Part of #2428.
|
| | | | |
|
| | | | |
|