| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
Fixes #935
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
As per #935.
I called this "circparameters_from_netparameters" not
"circparameters_from_netparams" because the type is "NetParameters"
not "NetParams".
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
keymgr: Fix `wrong_key_type` test.
Closes #938
See merge request tpo/core/arti!1341
|
| |/ / / / / /
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
The string representation of `SshKeyAlgorithm` on `main` changed between
the time the tests were introduced and the time they were merged.
|
| |\| | | | |
| |_|/ / /
|/| | | |
| | | | |
| | | | | |
tor-hsclient: Handle handshake completion error correctly
See merge request tpo/core/arti!1336
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Thsi is a bit of a mouthful now, but it is comprehensible.
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1336#note_2917235
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
See
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1336#note_2917236
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We need to handle errors during the handshake. That also produces a
`tor_proto::Error` but we want to print a different message for that,
so let's rename this before adding the new variant.
|
| | | |_|/
| |/| |
| | | |
| | | | |
It was wrong where ti was.
|
| |\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | | |
keymgr: Add tests for ArtiNativeKeyStore
See merge request tpo/core/arti!1337
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This will enable future tests to optionally initialize the test key
store with some test keys.
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
Part of #895
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This fixes a bug which caused the function to always return a
type-erased `()` instead of the actual key!
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Resolve or defer most TODO HS items in tor-netdir.
See merge request tpo/core/arti!1310
|
| | | | | |
| | | | |
| | | | |
| | | | | |
The function this comment is asking us to write is `circtarget_from_pieces`.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
(Adding comments explaining that these errors are really
unlikely to occur, unless there is a bug in our code or in Rust's
time handling.)
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This issue with walking over the ring is an issue we really must
solve on the services timeframe, as is the one about looking only at
the rings for which a blinded ID is germane.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This makes them configurable, since we allow the user to override
any consensus parameter.
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | |_|/
| |/| |
| | | |
| | | |
| | | | |
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.
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | | |
arti_client: Move BadOnion from TorAddrError to ErrorDetail.
Closes #932
See merge request tpo/core/arti!1331
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This variant can never be constructed when parsing a TorAddr, so it
doesn't make sense to have it in TorAddrError: the use can never see
it unless they enable `error_detail`.
Not a semver break because the client feature is not yet stable.
Closes #932.
|
| |\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | | |
keymgr: Add support for x25519 keys.
Closes #900
See merge request tpo/core/arti!1321
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
The `Unsupported` variant is no longer needed now that we support
`KeyType::X25519StaticSecret`s.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This also fixes a bug which caused the function to always return a
type-erased `()` instead of the actual key!
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This previously just re-exported `ssh_key::Algorithm`.
However, we will need to support x25519 keys (which aren't supported by
the `ssh_key` crate), so we define our own enum for the key algorithm.
In addition to the `Algorithm` variants from `ssh_key`, our enum also
has an `X25519` variant.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
arti cfg tests: exception declaration improvements
See merge request tpo/core/arti!1330
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
It's used for declarations only, and they should all be here.
|