| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| |\ \
| |/
|/|
| |
| |
| |
| | |
Fix flaky `request_fail_ultimately` test
Closes #2318
See merge request tpo/core/arti!3598
|
| |/
|
|
|
|
|
|
|
|
|
|
|
| |
This commit fixes the flaky `request_fail_ultimately` in tor-dirserver,
which is flaky due to the operating system's handling of TCP RSTs, which
are generally detected stochastically and sometimes are not received
even after the entire response has been parsed.
This leads to tor-dirclient either returning a connection reset or a
truncated header error, depending on whether it successfully or
unsuccessfully reads zero bytes from the server.
Fixes #2318
|
| |\
| |
| |
| |
| |
| |
| | |
maint: Use 'C.UTF-8' locale for sorting `exclude_contributors.txt`
Closes #2313
See merge request tpo/core/arti!3588
|
| | | |
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Make all non-main APIs in arti experimental.
Closes #2284, #2299, and #419
See merge request tpo/core/arti!3586
|
| | | | |
|
| | | |
| | |
| | |
| | | |
These crates are "inherently experimental" and allowed to require it.
|
| | | |
| | |
| | |
| | | |
Closes #2299.
|
| | | |
| | |
| | |
| | | |
No semantic changes.
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
Or rather, un-rename it. It's no longer a stable thing, so it
doesn't need to have this particular name.
|
| | | |
| | |
| | |
| | |
| | |
| | | |
Only fn main() is now pub when experimental-api is disabled.
Closes #2284.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
We'll use this to enforce the opacity of the arti crate, once it is
more opaque.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
tor-netdoc: Remove remaining use of EP::Other
See merge request tpo/core/arti!3595
|
| |/ / /
| | |
| | |
| | |
| | |
| | | |
This commit replaces the last remaining uses of `EP::Other` in
`tor-netdoc`, which got removed in arti!3561 but was still made use of
in arti!3592 without causing a merge conflict.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Implement EncodedAuthCert and use it in poc for votes
See merge request tpo/core/arti!3592
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
The feature arrangements in tor-netdoc are getting to be in need of a
serious overhaul.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The structural confusion hazard is now discussed extensively in
torspec and referenced from the code.
The proposal has been implemented.
|
| | | | |
| | | |
| | | |
| | | | |
This probably isn't going to happen, but let's keep the idea in-tree.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Move the information which still seems relevant out of the plan
document.
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
As per doc/dev/notes/authcert-in-consensus.md.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The string so far is exposed already via whole_for_signatures.
It is unusual for a doc parser to need this, but embedded documents
can use this plus byte_position to get the original input text for
their part of the outer document.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
Roughly as per the proposal in `doc/dev/notes/authcert-in-consensus.md`.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We can use the real AuthCert now that it implements the parse2 traits.
The verification function is still used by poc's netstatus code and by
a test case.
We must change the field names in a few places, because the real
AuthCert's struct field names are the keywords, whereas the poc's are
the key names. (A shame that they're different!)
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Use accessors for the body.
(Eventually this function will replace, be replaced by, or merge with,
the existing signature code outside poc.)
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Implement authority certificate management
See merge request tpo/core/arti!3561
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Adding a comment suggesting to implement this as part of abstracting
`Sha256` behind a more generic identifier.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This commit rpelaces the use of `INSERT` with `INSERT OR REPLACE` for
handling insertion conflicts in `store_insert`.
Because everything is content-addressed anyways, it does not matter to
do this, because if we get a conflict on a SHA256, it means the data is
equal.[1]
[1]: Excluding SHA-2 collisions which are impractical as of 2026.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This commit modifies the database::store_insert logic to accept an
iterator of the encodings to store the document in, instead of encoding
it with all encodings we support.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This commit changes `ErrorProblem::Other` to
`ErrorProblem::OtherBadDocument` while adding two new variants:
* `ErrorProblem::Internal`
* `ErrorProblem::BadApiUsage`
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Instead of failing, add them to missing and query them again.
|
| | | | | | |
|