aboutsummaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | tor-dirserver: Remove `preferred` from dataClara Engler2026-03-091-9/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit removes the `preferred` member field from the `Unverified` and `Verified` variant in `ConsensusBoundData` while adding it as a parameter to `StaticEngine::execute`, with the idea being that the retry logic is handled by the caller anyways, involving the selection of authorities. Right now, I am still a bit unsure how this will play out.
* | | | tor-dirserver: Fix rustdoc commentsClara Engler2026-03-092-5/+5
| | | |
* | | | tor-dirserver: Implement consensus loadingClara Engler2026-03-092-16/+347
| | | | | | | | | | | | | | | | | | | | | | | | This commit implements the consensus loading mechanism by glueing together the logic from the database module with regard to querying missing descriptors.
* | | | tor-dirserver: Query missing descriptor logicClara Engler2026-03-099-1/+656
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit implements the logic for querying missing descriptors by adding three new methods to ConsensusMeta: * missing_servers * missing_extras * missing_micros All of them essentially work the same, namely by querying the consensus_router_descriptor_member database table alongside the docid of the current consensus, left joining the respective router_descriptor and returning the digests on all rows where the left join resulteed in a NULL. A small exception are extra-info descriptors. There, we can only return the ones where we have an accompanying server descriptor, hence why we perform an inner join with the server descriptors between the from and the left join.
* | | | tor-dirserver: Rename `md` to `microdesc`Clara Engler2026-03-091-2/+2
| | | | | | | | | | | | | | | | | | | | This is better because it is returned by `ConsensusFlavor::Microdesc::name()`.
* | | | tor-dirserver: Fix extra info schemaClara Engler2026-03-091-7/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit fixes some parts in the extra-info related part of the database schema. Most notably, it changes the semantic to indicate that the sha1 represents the unsigned part of the extra-info document. Likewise, it also ensures that the reference to an extra-info document must not be null with a plain consensus but null with a microdescriptor consensus.
* | | | tor-dirserver: Store unsigned SHA for descsClara Engler2026-03-091-12/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit modifies the database schema to rename `sha1` and `sha2` to `unsigned_sha1` and `unsigned_sha2`. Network status consensuses refer to server descriptors by their unsigned sha1. Microdescriptor consensuses refer to microdescriptors by their sha2, which is always unsigned, so we adapt that name for consistency.
* | | | tor-dirserver: Remove foreign key from descriptor memberClara Engler2026-03-091-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit removes the foreign key constraints from the `consensus_router_descriptor_member` in order to allow the insertion of the router descriptors contained in a consensus before the respective descriptors were fetched. This also allows to directly compute the missing descriptors in SQL, using a left join on this table on `router_descriptor` and obtaining the entries that are NULL.
* | | | tor-dirserver: Store sha1 for extra-infoClara Engler2026-03-091-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit modifies the database schema by tracking the sha1 of the extra-info instead of the rowid. This makes generating queues and other things easier. It also removes the foreign key constraint in order to allow for an asynchronous retrieval and storage. Otherwise it would not be possible to store a router descriptor without having obtained the extra info first.
* | | | tor-dirserver: Use getters for meta structsClara Engler2026-03-092-13/+17
| | | | | | | | | | | | | | | | | | | | | | | | This approach is better in order to guarantee that invariants are not violated, such that the expiry timestamp not being earlier than the valid after one for example.
* | | | tor-dirserver: Derive Copy on database meta structsClara Engler2026-03-091-2/+2
| | | |
* | | | tor-dirserver: Implement Sha1Clara Engler2026-03-092-4/+12
| | | | | | | | | | | | | | | | | | | | This commit implements Sha1 for the database in order to use it for the fingerprints in authority key certificates.
* | | | tor-dirserver: Implement Sha3_256Clara Engler2026-03-092-8/+6
| | | | | | | | | | | | | | | | | | | | This commit implements Sha3_256 as a database type and uses it for stroing `ConsensusMeta::unsigned_sha3_256`.
* | | | tor-dirserver: Generic database hash implementationClara Engler2026-03-092-65/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit moves the `DocumentId` implementation into a macro called `impl_hash_wrapper` that implements a hash type in a database compatible fashion. In our case, we implement this for `sha2::Sha256` and then type alias `DocumentId` to this new hash. Yes, we originally moved away from a type alias here, but I think this is fine because for the foreseeable future, we will continue to use a hash here, just maybe not Sha2, but the flexibility remains. The motivation for this is to support other hash algorithms similarly too.
* | | | tor-dirserver: pub(crate) for ConsensusMeta::lifetimeClara Engler2026-03-091-1/+1
| | | |
* | | | tor-dirserver: Move AuthCert to databaseClara Engler2026-03-092-573/+263
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit moves get_recent_auth_certs from operation to database by introducing a new struct called `AuthCertMeta` containing the database metadata alongside an accompanying data method returning the raw data. For now, it leaves out the download, verify, and insert logic. We will add that back later once we will need it.
* | | | tor-dirserver: Rename Consensus to ConsensusMetaClara Engler2026-03-092-27/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit renames the database `Consensus` to `ConsensusMeta` in order to not collide with the naming from tor-netdoc and to clearly indicate that this data is just metadata about such a document, but not the document itself.
* | | | tor-dirserver: Use database as dbClara Engler2026-03-091-4/+4
| | | | | | | | | | | | | | | | Makes things more handy to write.
* | | | tor-dirserver: Avoid super in testsClara Engler2026-03-091-54/+44
| | | | | | | | | | | | | | | | | | | | If is annoying to use, especially if we can avoid it by just avoid name conflicts.
* | | | tor-dirserver: `use crate::database::Consensus`Clara Engler2026-03-091-3/+3
| | | |
* | | | tor-dirserver: Move db constants to the topClara Engler2026-03-091-172/+172
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Right now, it is a bit unfortunate that we have the very long schema string between two code blocks. It is probably better to either have it at the top or the bottom of the file, hence why this commit moves it to the top.
* | | | tor-dirserver: Move sync timeout to databaseClara Engler2026-03-092-55/+38
| | | | | | | | | | | | | | | | | | | | This commit moves calculate_sync_timeout into the Consensus struct in the database module, as it fits better there.
* | | | tor-dirserver: Remove serve functionClara Engler2026-03-092-129/+2
| | | | | | | | | | | | | | | | | | | | This commit removes the operation serve function because it no longer fits into the new model of operation using a FSM.
* | | | tor-dirserver: Refactor consensus retrieval logicClara Engler2026-03-092-260/+324
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit moves the get_recent_consensus logic to the database module, which also introduces a struct querying all fields in it. This not only simplifies the return type but also makes working with this type more comfortable to work with.
* | | | tor-dirserver: docid as consistent foreign keyClara Engler2026-03-091-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | This commit replaces occurrences of rowid with docid, particularly in tables for N:M cardinalities. Purpose of this is, to achieve a greater consistency with the overall database schema.
* | | | tor-dirserver: Return Timestamp in consensus queryingClara Engler2026-03-091-17/+10
| | | | | | | | | | | | | | | | | | | | This is more useful and less boilerplate, removing a parameter for a conversion we can do ourselves if required.
* | | | tor-dirserver: Scratch out the FSMClara Engler2026-03-092-5/+493
| |/ / |/| | | | | | | | | | | | | | | | | This commit scratches out the FSM for the dirmirror operation. Right now, there are still lots of TODO, lots of code warnigns, and such. It should model the rough concept.
* | | Merge branch 'nonblocking-rpc-part3' into 'main'Nick Mathewson2026-03-057-213/+1007
|\ \ \ | | | | | | | | | | | | | | | | rpc: API for event-driven IO. See merge request tpo/core/arti!3652
| * | | rpc: edit and revise EventLoop documentation.Nick Mathewson2026-03-051-7/+8
| | | |
| * | | rpc: Transpose Results in RpcPoll::poll.Nick Mathewson2026-03-051-7/+5
| | | |
| * | | rpc: document when try_as_{fd,socket} can fail.Nick Mathewson2026-03-051-1/+11
| | | |
| * | | rpc: Usage documentation for polling APIIan Jackson2026-03-053-22/+168
| | | |
| * | | rpc: Propose to transpose a Result<Result<>>Ian Jackson2026-03-051-0/+2
| | | |
| * | | rpc: Add a question in the form of a TODOIan Jackson2026-03-051-0/+2
| | | |
| * | | rpc: Fix many long lines in docs and commentsIan Jackson2026-03-054-22/+44
| | | |
| * | | rpc: Fix two mis-formatted json! blocksIan Jackson2026-03-051-2/+13
| | | | | | | | | | | | | | | | | | | | | | | | rustfmt doesn't like to touch things inside macros. These were misformatted as a result, with long lines and in one case a missing space.
| * | | rpc: Remove WantIo and expose wants_to_write().Nick Mathewson2026-03-053-43/+21
| | | |
| * | | rpc: Change Waker implementation to EventLoop.Nick Mathewson2026-03-056-112/+150
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Per discussion, it makes more sense to have the API be one that gets called when our IO interests change. Additionally, this commit removes the try_reading and try_writing booleans, as previously discussed. I've left a couple of XXXX comments where more documentation or thought is likely needed.
| * | | rpc: API for event-driven IO.Nick Mathewson2026-03-055-21/+245
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This API provides the necessary functionality to use an RpcConn inside a poll-like event loop. This is part of #1856.
| * | | RPC: Explain why UserTag is a separate type.Nick Mathewson2026-03-051-0/+8
| | | |
| * | | RPC: Raname Tag,RequestTag to UserTag.Nick Mathewson2026-03-053-37/+33
| | | |
| * | | rpc: documentation fixes and clarificationsNick Mathewson2026-02-261-3/+8
| | | | | | | | | | | | (from Diziet)
| * | | rpc: Add a paragraph to the connimpl overviewNick Mathewson2026-02-261-0/+3
| | | |
| * | | rpc: Add overview documentation to connimpl.Nick Mathewson2026-02-261-0/+64
| | | | | | | | | | | | | | | | This is based on a pad with input from nickm and Diziet
| * | | rpc: Nonblocking request submit API (Rust only)Nick Mathewson2026-02-262-5/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit finally adds a `RpcConn::submit()` method to send a request without having to wait on it specifically, and an `RpcConn::wait()` method to wait for the next response from _any_ such request. Thanks to the previous patches, this is relatively simple! This is part of #1856.
| * | | rpc: Backend support for pollable requests.Nick Mathewson2026-02-263-7/+92
| | | | | | | | | | | | | | | | | | | | | | | | In order to implement tagged pollable requests, we need a separate response queue for them, and we need to dispatch requests to that queue as appropriate.
| * | | rpc: Add "Tag"s to response queues.Nick Mathewson2026-02-261-25/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | For pollable requests, we'll want to associate each one with a tag. But we'd rather not carry tags around for _every_ pending request: it would waste space and lead to possible errors. So instead we add a Tag type as a member of QueueId.
| * | | rpc: Refactor connimpl in preparation for pollable requests.Nick Mathewson2026-02-263-59/+189
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to implement this part of #1856, we will internally divide requests into two kinds: "Waitable" and "Pollable". Waitable requests are the kind that we have now: They are created with an "execute" method. They each have their own response queue and their own condvar, and in order to see if they have any responses, the caller needs to call some kind of request-specific method. Pollable requests are the ones we will add. They are created with a "submit" method, and associated with a user-provided tag. They all share the same queue and the same condvar. To see if any of them have a response, the caller will run a function that returns tagged responses. In order to support this division, this commit: - turns `RequestState` into an enum, - makes `ResponseQueue` into its own type, - Adds a trait that will be implemented by every type that can identify a response queue.
* | | | Merge branch 'upgrade-polyval' into 'main'Nick Mathewson2026-03-052-3/+10
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | proto: Upgrade to latest polyval. Closes #2390 See merge request tpo/core/arti!3747
| * | | proto: Upgrade to latest polyval.Nick Mathewson2026-03-042-3/+10
| | | | | | | | | | | | | | | | | | | | | | | | This will improve performance for CGO. Closes #2390.