summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | tor-netdoc: Drop the allow(unused)Ian Jackson2025-12-101-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Now we use all the imports. We're still adding much code, though.
| * | | | | | tor-netdoc: recast Ignored and impl new encoding traitsIan Jackson2025-12-102-5/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a bit subtle. Ignored is used where we want to see and accept a value (for example, an Item or an Object), but we don't want to emit it. Update the docs and implement the various traits. We need an uninhabited helper type which implements ItemValueEncodable and ItemObjectEncodable but *not* ItemArgument. That prevents the use of Ignored for an argument, which would be wrong because what would we emit to keep the argument parser in step?
| * | | | | | tor-netdoc: encoding: impl new traits for NotPresentIan Jackson2025-12-101-0/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Per the type-level docs for NotPresent: it's is a placeholder type mostly used when the document variety doesn't have this field. When encoding it, we emit nothing.
| * | | | | | tor-netdoc: encoding: Provide multiplicity moduleIan Jackson2025-12-103-1/+214
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is quite like the one for parse2, but separate for Reasons. Also add a bit of explanation to parse2's multiplicity.rs, about why there are all these different FooSetMethods traits, which the encoder doesn't need.
| * | | | | | tor-netdoc: encoding: Add new traits that we will deriveIan Jackson2025-12-101-0/+99
| | | | | | | | | | | | | | | | | | | | | | | | | | | | And implement them for Void and tuples, as applicable.
| * | | | | | tor-netdoc: Add imports to encode module, for new encode deriveIan Jackson2025-12-102-2/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding these all at the beginning helped me with churn in this branch. The allow(unused) will go away again shortly.
| * | | | | | tor-netdoc: NetdocParseable: Document that BTreeSet is allowedIan Jackson2025-12-102-0/+5
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we implemented this we documented it for ItemValueParseable but forgot to do so for NetdocParseable. Add some xrefs to the traits to try to stop similar mistakes.
* | | | | | Merge branch 'dirmirror-download' into 'main'Clara Engler2025-12-106-3/+438
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | tor-dirserver: Implement download functionality See merge request tpo/core/arti!3434
| * | | | | tor-dirserver: Document nested round-robinsClara Engler2025-12-101-2/+15
| | | | | |
| * | | | | tor-dirserver: Improve download_single's docClara Engler2025-12-101-5/+2
| | | | | |
| * | | | | tor-dirserver: Rename DownloadManager to ConsensusBoundDownloaderClara Engler2025-12-101-13/+13
| | | | | |
| * | | | | tor-dirclient: Remove Downloadable for goodClara Engler2025-12-102-28/+4
| | | | | |
| * | | | | tor-dirclient: Fix Requestable rustdocClara Engler2025-12-101-2/+2
| | | | | |
| * | | | | tor-dirserver: Add doc for TcpConnect errorClara Engler2025-12-101-0/+4
| | | | | |
| * | | | | tor-dirserver: Documentation improvementsClara Engler2025-12-101-3/+5
| | | | | |
| * | | | | tor-dirserver: Replace IO with TcpConnect errorClara Engler2025-12-102-4/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit replaces `AuthorityCommunicationError::IO` with `AuthorityCommunicationError::TcpConnect` while removing the respective `From<std::io::Error>` implementation, mostly because an IO error is way too generic and in our case, it would only occurr with a failed TCP connection.
| * | | | | tor-dirclient: Improve upon Requestable subtraitsClara Engler2025-12-101-11/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit removes the unused `Votable` and `Uploadable` traits and changes the `Downloadable` trait to depend on `Requestable` rather the sealed type, as it achieves the same level of sealedness while clearly communicating the hierarchy.
| * | | | | tor-dirserver: Fix wrong warning sectionClara Engler2025-12-101-1/+1
| | | | | |
| * | | | | tor-dirserver: Implement DownloadManagerClara Engler2025-12-105-0/+405
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit implements the DownloadManager in its own module, an object responsible for downloading network documents from an upstream directory authority. Further motivation for this can be found in the rustdoc comment of the respectively introduced type.
| * | | | | tor-dirclient: Semantic traits for RequestableClara Engler2025-12-101-0/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit introduces three new traits to `tor-dirclient`: * `Downloadable` * `Uploadable` * `Votable` All of these traits have no associated methods or constants and all require an existing implementation of `Requestable`. The idea behind this is for applications to accept generic arguments based upon the semantic meaning, allowing for a better integration with proposal330.
| * | | | | tor-dirclient: Improve upon error loggingClara Engler2025-12-101-3/+3
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | This commit improves upon the error logging of the `tor-dirclient` crate in order to not just log the location/source of an error but also the actual reason of an error.
* | | | | Merge branch 'relay-or-config' into 'main'opara2025-12-092-4/+65
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | arti-relay: Advertise config fields are now lists See merge request tpo/core/arti!3538
| * | | | | arti-relay: advertise config fields are now listsSteven Engler2025-12-092-4/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Example: ```toml [relay.advertise] ipv4 = [] ipv6 = ["[10::]:10"] ``` ```text Error: Failed to resolve configuration [...] The list is empty for key "default.relay.advertise.ipv4" in [...] ```
* | | | | | Merge branch 'configure-deftly-v5' into 'main'Nick Mathewson2025-12-0911-113/+3326
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement derive_deftly(TorConfig) Closes #1704 See merge request tpo/core/arti!3513
| * | | | | | tor-config: Make derive(Flattenable) use $crate.Nick Mathewson2025-12-091-4/+3
| | | | | | |
| * | | | | | Revert "tor-config: Code to serialize Option<Option<Duration>> via humantime"Nick Mathewson2025-12-093-28/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit c7c8eb5a03439ce31319389183e64cb0db539fc9.
| * | | | | | Demonstration: convert tor-circmgr to use derive_deftly(TorConfig)Nick Mathewson2025-12-091-98/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (We could tighten this up even more by inlining some of the default values.)
| * | | | | | tor-config: new derive_deftly(TorConfig) template.Nick Mathewson2025-12-094-0/+3044
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This template is meant to replace most of our use of derive_builder for configuration objects. Where possible and reasonable, it delegates to existing macros, and automatically infers what special patterns we use for individual types. In other cases, it uses compile-time errors to inform the caller about pattern violations.
| * | | | | | tor-config: New Buildable trait.Nick Mathewson2025-12-091-2/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This trait will be implemented by every type that our derive_deftly(TorConfig) template generates. It will, among other things, help us figure out the Builder type for a given config type in cases where string-pasting magic is intractable, or where we want to use assert_not_impl to double-check the attributes.
| * | | | | | tor-config: Helper types for setters to take as argumentsNick Mathewson2025-12-092-0/+135
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The derive-deftly TorConfig template will use these as appropriate for the inputs to setter functions, based on field types.
| * | | | | | tor-config: Code to serialize Option<Option<Duration>> via humantimeNick Mathewson2025-12-093-0/+28
| | | | | | |
| * | | | | | tor_config: Make ListBuilders implement ExtendBuilder.Nick Mathewson2025-12-091-0/+11
| | | | | | |
| * | | | | | tor-config: Give Flattenable template a location.Nick Mathewson2025-12-091-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we don't do this, then any attempt to use it from a derive-deftly template will cause a warning about referring to it as $crate::macroname.
| * | | | | | tor-config: Allow Flattenable to apply to types with generics.Nick Mathewson2025-12-091-1/+2
| | | | | | |
| * | | | | | map_builder: Alternative syntax for use by derive(TorConfig)Nick Mathewson2025-12-091-5/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | derive_deftly wants to expand types before passing them to macro_rules macros, which is quite reasonable. But map_builder wants its input collection type to be an `ident`, not a `path`. (And macro_rules doesn't accept `path` before a `<`.) To fix this, we're providing an alternative syntax for map_builder, where the inputs are the map type and the builder map type.
| * | | | | | extend_builder: Implement ExtendBuilder for HashMap.Nick Mathewson2025-12-091-1/+17
| | | | | | |
| * | | | | | tor-config: missing #[allow(dead_code)] in derive_list_builder_helperNick Mathewson2025-12-091-0/+1
| | | | | | |
| * | | | | | tor-config: missing $crate in derive_list_builder_helperNick Mathewson2025-12-091-1/+1
| | | | | | |
| * | | | | | tor-config: obviate need for Educe in derive_list_builder_helperNick Mathewson2025-12-091-2/+11
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We were only using it for Default, and it is easier to implement Default by hand than it is to keep this particular layer in our leaning tower of macros.
* | | | | | Merge branch 'fix-relay-compile' into 'main'David Goulet2025-12-095-9/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | proto: Fix relay/hs-service feature gating See merge request tpo/core/arti!3534
| * | | | | | proto: Remove outdated TODO about add_ent_with_id()Gabriela Moldovan2025-12-081-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We now have add_ent_with_id(), so we can just remove the TODO.
| * | | | | | proto: Fix relay/hs-service feature gating (fmt)Gabriela Moldovan2025-12-081-1/+1
| | | | | | |
| * | | | | | proto: Fix relay/hs-service feature gatingGabriela Moldovan2025-12-085-6/+14
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | Without this, `tor-proto` doesn't compile if you enable the `relay` feature but not `hs-service`.
* | | | | | Merge branch 'check-licenses' into 'main'Ian Jackson2025-12-091-6/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | check_licenses: Update comment about priority-queue See merge request tpo/core/arti!3509
| * | | | | | check_licenses: Adjust comment on priority-queue exceptionGabriela Moldovan2025-12-091-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Technically `priority-queue` belongs in the `MPL_20_OK` list, but our current checks are too rudimentary to be able to handle the `OR` (we *do* split on `OR` for one of the checks, but we only compare the resulting licenses against `RECOGNIZED_LICENSES`, not `MPL_20_OK`). So for now, let's retain the exception for `priority-queue`. (We could expand the logic in the branch handling the `OR'`d licenses to fix this issue, but the result would be quite fragile; and besides, I think we're going to eventually want to replace this with a proper license parser).
| * | | | | | check_licenses: Update comment about priority-queueGabriela Moldovan2025-12-091-2/+0
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They have in fact added the MPL "Exhibit A" text to every file in the crate. The license is still LGPL-3.0-or-later OR MPL-2.0, so the rest of the comment still applies. See https://github.com/garro95/priority-queue/pull/54/commits/e2f90b0811c252f80165bbdb19233464289e428f
* | | | | | Merge branch 'fix-32bit-builds' into 'main'opara2025-12-081-4/+10
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | tor-memquota: fix builds for 32-bit platforms See merge request tpo/core/arti!3532
| * | | | | tor-memquota: fix builds for 32-bit platformsYaksh Bariya2025-12-091-4/+10
|/ / / / / | | | | | | | | | | | | | | | | | | | | Multiplying 8 * GIB overflows the usize limit (4 GIB - 1) on 32-bit platforms. So handle this properly for 32-bit platforms
* | | | | Merge branch 'mistune' into 'main'opara2025-12-082-13/+138
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | maint: Fix 'extract-md-links' to work with mistune 3.1.3 See merge request tpo/core/arti!3517
| * | | | | maint: fix changelog link orderSteven Engler2025-12-081-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | This is to make `update-md-links --check CHANGELOG.md` pass.
generated by cgit v1.2.3 (git 2.46.0) at 2026-08-27 21:53:56 +0000