| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | | | | | |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
The removed tests are ones that no longer make sense given our
simplified behavior.
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | |/ / / /
| |/| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
We no longer plan to have all weak-refs automatically de-duplicated,
for reasons discussed at #868. This lets us discard a lot of code,
including code to check for Arc/weak identity.
I've disabled some unit tests here, but I'll add them back or
rewrite them (or discard them as irrelevant) by the end of the branch.
|
| | |_|/ / /
|/| | | |
| | | | |
| | | | |
| | | | | |
This updates some outdated references from back when `derive-deftly` was
called `derive-adhoc`.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Prompted by
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3891#note_3395849
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This updates a doc and the corresponding test.
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This also makes the macro `beta_deftly`.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
I want to make all uses of `keypair_specifier` unquoted, so this can't
be a `token_stream` (and in fact, `keypair_specifier` was always meant
to be a type).
|
| | |/ / /
|/| | |
| | | |
| | | | |
This attribute is called `keypair_specifier`, not `key_specifier`.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
proto: Add relay channel handshake unit tests
See merge request tpo/core/arti!3853
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: David Goulet <[email protected]>
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The CreateRequestHandler was introduced prior to the relay handshake
unit tests so fix them now that we have a way to build one.
Signed-off-by: David Goulet <[email protected]>
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We can't access `NetParameters` in this crate to build from consensus
default value so instead add a `defaults_for_tests()` gated function for
each parameters object which puts in the defaults from the spec.
We'll need CircNetParameters in order to build create request handler
for unit tests hence why it is in src/circuit.rs.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
From the reactor.rs to channel.rs, makes more sense and we'll need it in
the handshake tests.
No behavior change, just code movement and a function rename.
Signed-off-by: David Goulet <[email protected]>
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: David Goulet <[email protected]>
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Instead of copying the client unit tests into the channel module, just
make both current unit tests run on a client and relay handshake.
This required a bit of trickery with type HandshakeConnectFn but works
out in the end.
It also adds the RelayMsgBuf that wraps a MsgBuf in order to implement
CertifiedConn which is very relay only.
Signed-off-by: David Goulet <[email protected]>
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Just for mental sanity to have all const at the same place and easily
readable.
No behavior change.
Signed-off-by: David Goulet <[email protected]>
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
No behavior change.
Make the const cells public as they will be useful for more upcoming
unit tests especially on the relay side.
Signed-off-by: David Goulet <[email protected]>
|
| |\ \ \ \ \
| |/ / / /
|/| | | |
| | | | |
| | | | | |
arti-relay: Log the public ntor key
See merge request tpo/core/arti!3882
|
| | | | | |
| | | | |
| | | | |
| | | | | |
We only need the max, not the entire sorted list.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This is useful for testing.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Small cleanup.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Code movement only.
|
| | |/ / /
| | | |
| | | |
| | | | |
Set up function placeholder.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
tor-proto: Fix stream XON/XOFF flow control sidechannel mitigation bug
Closes #2400
See merge request tpo/core/arti!3884
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
If this is zero, then it means that the consensus had bad values.
But may as well check it anyways.
|
| | | | | | |
|
| | | | | | |
|
| | |/ / / |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
tor-netdoc: parse2: Fix error for invalid item argument
See merge request tpo/core/arti!3879
|
| | |/ / /
| | | |
| | | |
| | | | |
When we can't parse a NormalItemArgument, that's Invalid, not Missing.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
I also searched this file for FromStr impls and these types were the
only ones that ought to be NormalItemArgument but weren't.
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Unfortunately one derive can't invoke another, so we must open-code
them, just like the other traits.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Nothing derives just Transparent yet.
Mostly code motion, with some additional furniture and docs.
Review with git show --color-moved.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This error message is going to be used by both Transparent and
BytesTransparent. Put it in the code that is going to be part of
Transparent, and remove the macro name from it.
|
| | | | | |
|