| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
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.
|
| | | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | | |
As per discussion in
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1320#note_2916689
and IRC followup.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Resolve a few "TODO HS" items in circmgr.
See merge request tpo/core/arti!1322
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
These two functions are only slightly different, and benefit from
taking a Fn.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
I looked through the C tor source code and couldn't find any
additional path restrictions.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
One of these is test-related; one is vanguards-related.
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Documentation, mostly on channel and circuit lifecycles
See merge request tpo/core/arti!1318
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Adjust formatting, add more references to lower-level APIs, and
clarify lifetimes a little more.
|
| | | | | | | |
|
| | | | | | | |
|
| | |/ / / / |
|
| |\ \ \ \ \
| |/ / / /
|/| | | |
| | | | |
| | | | | |
Remove spurious todo-hs items in tor-cert.
See merge request tpo/core/arti!1311
|
| |/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
I am not sure why we wrote these comments, but they are incorrect:
I've investigated the C code and found only 3 key types. The
"unimplemented" types that the TODO comment here complains about are
in fact certificate types.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
keymgr: Downgrade "TODO hs" to "TODO HSS".
See merge request tpo/core/arti!1334
|
| | | |_|/
| |/| |
| | | |
| | | |
| | | | |
These TODOs can be deferred for now: we're not declaring the keymgr APIs
stable until we add support for hidden services.
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
tor-hsclient: Make RendPtIdentityForError use RelayIds
See merge request tpo/core/arti!1335
|
| | | | |
| | | |
| | | |
| | | | |
Since arti!1233 this is trivial.
|
| |/ / /
| | |
| | |
| | |
| | | |
For some reason this wasn't used in the actual errors,
although it *was* used in the types of the functions etc.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
tor-hsclient: Discuss intro circuit extend-and-reuse
See merge request tpo/core/arti!1298
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
This is a summary of a discussion I had on IRC.
|