summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | RelayFlags: Use the singular RelayFlag for naming individual flagsIan Jackson2025-11-1811-47/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're going to separate RelayFlag from RelayFlags. We could continue to provide a bunch of constant values for RelayFlags, for each individual type, but that would involve an ad-hoc derive, and would still be a bit of an unusual API. Instead, we're going to make ``RelayFlag` a normal enum. In this commit, we change everyone to refer to it by its singular name. Hopefully separating out this bulk change makes both this, and the core commit, easier to review.
| * | | | | | | tor-netdoc: rs builder: Take Into<RelayFlags>Ian Jackson2025-11-181-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Many call sites pass one flag. We're going to have a type-level distinction between a single flag and a set. But that's annoying at many of the call sites which want to specify a fixed single flag. Note that `set_flags` doesn't only set flags. It replaces them.
| * | | | | | | tor-netdoc: RelayFlags: Tidy up from_item for flagsIan Jackson2025-11-182-3/+5
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement this on DocRelayFlags instead, since that's what it returns. And change the name to from_item_consensus to highlight its semantic restriction. (It's not generic over the implied flags, so it can only do one thing.)
* | | | | | | Merge branch 'upgrade-bigint' into 'main'Nick Mathewson2025-11-181-3/+2
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | maint: Upgrade num-bigint-dig to 0.8.6 Closes #2110 See merge request tpo/core/arti!3475
| * | | | | | maint: upgrade num-bigint-dig to 0.8.6Steven Engler2025-11-181-3/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This appears to resolve the warning: > warning: the following packages contain code that will be rejected by a future version of Rust: num-bigint-dig v0.8.4
* | | | | | Merge branch 'p2-relayflags' into 'main'Ian Jackson2025-11-1827-221/+700
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | tor-netdoc: Overhaul RelayFlags, and provide for conditionally retaining unknown info See merge request tpo/core/arti!3454
| * | | | | tor-netdoc: Add an explanation about RelayFlags's PartialEqIan Jackson2025-11-181-0/+13
| | | | | |
| * | | | | tor-netdoc: Rename relay flags omitted/implicit const params (rustfmt)Ian Jackson2025-11-181-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | Some of the new formatting is a bit unfortunate. Ah well.
| * | | | | tor-netdoc: Rename relay flags omitted/implicit const paramsIan Jackson2025-11-181-16/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3454#note_3288967
| * | | | | tor-netdoc: semver.md updates for RelayFlags overhaulIan Jackson2025-11-181-0/+6
| | | | | |
| * | | | | tor-netdoc: Be able to retain unknown flagsIan Jackson2025-11-184-11/+25
| | | | | |
| * | | | | tor-netdoc: Introduce Unknown and retain_unknown_values optionIan Jackson2025-11-184-3/+143
| | | | | |
| * | | | | tor-netdoc: Add experimental retain-unknown featureIan Jackson2025-11-181-0/+4
| | | | | |
| * | | | | tor-netdoc: parse2: Introduce ParseOptions (fmt)Ian Jackson2025-11-181-1/+5
| | | | | |
| * | | | | tor-netdoc: parse2: Introduce ParseOptionsIan Jackson2025-11-183-8/+63
| | | | | | | | | | | | | | | | | | | | | | | | There are currently no options.
| * | | | | tor-netdoc: parse2: Introduce ParseInput intermediate structIan Jackson2025-11-187-26/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're going to add options arguments, effectively, to the parsing entrypoints. To avoid a proliferation of entrypoints (eg, parse_multiple_with_options), encapsulate the run-time input values in this structure.
| * | | | | tor-netdoc: Remove the RelayFlags type alias in netstatusIan Jackson2025-11-188-11/+11
| | | | | | | | | | | | | | | | | | | | | | | | I would do `#[deprecated]` but empirically that has no effect...
| * | | | | tor-netdoc: Introduce DocRelayFlags for relay flags in a network documentIan Jackson2025-11-186-12/+39
| | | | | | | | | | | | | | | | | | | | | | | | This will have space for unknown flags.
| * | | | | tor-netdoc: Make the weird parser not be a FromStr implIan Jackson2025-11-181-11/+9
| | | | | | | | | | | | | | | | | | | | | | | | This is the minimal change to provide sanity.
| * | | | | tor-netdoc: Improve and expose RelayFlagsParserIan Jackson2025-11-181-7/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently the only exposed API for parsing RelayFlags is the FromStr impl which is very strange. I'm going to remove it. But there should be a replacement. So improve RelayFlagsParser: * Derive Debug and Clone. * Replace the `&'static str` error. * Make sure unknown `IMPLIED | IMPLICIT` is detected at compile time. * Make it pub.
| * | | | | tor-netdoc: Separate ConsensusRelayFlagsParser and VoteRelayFlagsParserIan Jackson2025-11-183-10/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Provide a per-variety alias and use it for parsing; no longer impl ItemValueParseable for RelayFlags.
| * | | | | tor-netdoc: Paramterise RelayFlagsParser for varying omitted/implied bitsIan Jackson2025-11-181-5/+28
| | | | | | | | | | | | | | | | | | | | | | | | Which bits are implied/omitted depends on the document!
| * | | | | tor-netdoc: Introduce a RelayFlagsBits type aliasIan Jackson2025-11-181-1/+4
| | | | | |
| * | | | | tor-netdoc: Test RelayFlags coherence and encodingIan Jackson2025-11-181-0/+41
| | | | | |
| * | | | | tor-netdoc: Provide for encoding RelayFlagsIan Jackson2025-11-181-1/+22
| | | | | | | | | | | | | | | | | | | | | | | | The API is a bit clumsy, but we're mostly going to use it internally.
| * | | | | tor-netdoc: RelayFlags: add a macro for keyword parsingIan Jackson2025-11-181-19/+43
| | | | | |
| * | | | | tor-netdoc: Rename RelayFlags elements for regularity vs specIan Jackson2025-11-186-12/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add underscores so that the spec keywords and the flag constants correspond (according to `paste`'s case-changing rules).
| * | | | | tor-netdoc: Improve docs for RelayFlagsIan Jackson2025-11-181-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Document the bizarre FromStr impl. That isn't in my way, so I don't intend to fix it. So I've documented it instead. * Mention the naming discrepancy.
| * | | | | tor-netdoc: Move RelayFlags into its own moduleIan Jackson2025-11-183-161/+185
|/ / / / / | | | | | | | | | | | | | | | | | | | | We're going to do a lot more with this and it's less confusing to have it all together.
* | | | | Merge branch 'circ-hops' into 'main'David Goulet2025-11-1816-521/+807
|\ \ \ \ \ | |_|_|_|/ |/| | | | | | | | | | | | | | proto: Split CircHop into inbound and outbound components See merge request tpo/core/arti!3470
| * | | | proto: Fix up doc links post-refactoringGabriela Moldovan2025-11-172-9/+9
| | | | |
| * | | | proto: Temporarily silence a few warningsGabriela Moldovan2025-11-171-0/+3
| | | | |
| * | | | proto: Use CircHopInbound in the ForwardReactor (fmt)Gabriela Moldovan2025-11-173-6/+4
| | | | |
| * | | | proto: Use CircHopInbound in the ForwardReactorGabriela Moldovan2025-11-172-19/+15
| | | | |
| * | | | proto: Use CircHopOutbound in BackwardReactorGabriela Moldovan2025-11-172-34/+24
| | | | |
| * | | | proto: Make cc state shared between CircHop{Inbound,Outbound}Gabriela Moldovan2025-11-173-27/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The relay reactor needs the cc state to be shared between the inbound and outbound components (i.e. the forward and backward reactors), so we need to put the cc state behind a mutex. There will never be any contention on this mutex in the client impl, because the client reactor doesn't split the `CircHop` into `CircHopInbound` and `CircHopOutbound`. In the future, we should work on trying to reduce the number of locks in the `CircHop` states.
| * | | | proto: Make CongestionWindow CopyGabriela Moldovan2025-11-173-8/+8
| | | | |
| * | | | proto: Move CircHop{Inbound,Outbound} out of the client module.Gabriela Moldovan2025-11-172-575/+599
| | | | | | | | | | | | | | | | | | | | These will be soon used by relays too.
| * | | | proto: Add constructors for CircHop{Inbound,Outbound}Gabriela Moldovan2025-11-171-19/+46
| | | | |
| * | | | proto: Move some of CircHop's impl to CircHop{Inbound,Outbound}Gabriela Moldovan2025-11-171-141/+340
| | | | | | | | | | | | | | | | | | | | | | | | | The Inbound/Outbound CircHop states will be used in the relay reactor, so it's helpful to move some of the `CircHop` impl there.
| * | | | proto: Make the hop in SendRelayCell optional (fmt)Gabriela Moldovan2025-11-171-1/+5
| | | | |
| * | | | proto: Make the hop in SendRelayCell optionalGabriela Moldovan2025-11-177-20/+22
| | | | |
| * | | | proto: Update SendRelayCell docsGabriela Moldovan2025-11-171-2/+3
| | | | |
| * | | | proto: Move SendRelayCell to circhop (fmt)Gabriela Moldovan2025-11-174-5/+5
| | | | |
| * | | | proto: Move SendRelayCell to circhopGabriela Moldovan2025-11-175-20/+23
| | | | | | | | | | | | | | | | | | | | | | | | | This will be used by relays too, once we modify it to make the `HopNum` optional.
| * | | | proto: Split CircHop into inbound and outbound components (fmt)Gabriela Moldovan2025-11-171-7/+13
| | | | |
| * | | | proto: Split CircHop into inbound and outbound componentsGabriela Moldovan2025-11-171-52/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unlike the client reactor, the relay reactor uses these components in separate tasks. Splitting `CircHop` this way enables us to reuse its parts instead of duplicating them in the relay impl. Eventually, I'd like us to rewrite the client reactor to follow a similar pattern.
| * | | | proto: Rename cell decoder field for clarityGabriela Moldovan2025-11-141-3/+3
| | | | |
| * | | | proto: Avoid exposing stream map field (fmt)Gabriela Moldovan2025-11-141-8/+11
| | | | |
| * | | | proto: Avoid exposing stream map fieldGabriela Moldovan2025-11-141-3/+3
| | | | |
generated by cgit v1.2.3 (git 2.46.0) at 2026-08-27 14:53:44 +0000