| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | |
| | |
| | |
| | | |
These are in the spec and tor-hsclient is about to want them.
|
| |\ \ \
| |_|/
|/| |
| | |
| | | |
keymgr: Fix keymgr docs
See merge request tpo/core/arti!1343
|
| | | |
| | |
| | |
| | |
| | | |
`Error` is private to `dummy.rs` so let's not mention it in the docs of
the public items.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This fixes a rustdoc warning when building with:
```
cargo doc --locked --offline -p arti --no-default-features --features=full,onion-service-client
```
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
netdir: Conditionally expose APIs for working with unusable relays; document what "usable" means.
See merge request tpo/core/arti!1325
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Network-health wants these to see whether a given relay is listed in
the consensus.
cc @juga
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
tor-hsclient: Reimplement Debug for MockCirc
See merge request tpo/core/arti!1324
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This is less code. Also it now dumps the connect_called field which
was introduced in the meantime.
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | | |
tor-hsclient: Use proper CircParameters
Closes #935
See merge request tpo/core/arti!1340
|
| | | | |
| | | |
| | | |
| | | | |
Fixes #935
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
As per #935.
I called this "circparameters_from_netparameters" not
"circparameters_from_netparams" because the type is "NetParameters"
not "NetParams".
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
| | | | | |
|