| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| | * | proto: Fix docs to refer to the right stream:: paths. | Gabriela Moldovan | 2025-10-16 | 2 | -2/+2 | |
| | | | ||||||
| | * | proto: Move IncomingDataCmdChecker to top-level stream module. | Gabriela Moldovan | 2025-10-16 | 4 | -49/+55 | |
| | | | | | | | | | | | | | | | This will be used by relays too (for validating incoming messages on streams). This is just code motion, so it's best reviewed with `--color-moved`. | |||||
| | * | proto: Split out DataCmdChecker into multiple types (fmt). | Gabriela Moldovan | 2025-10-16 | 1 | -1/+1 | |
| | | | ||||||
| | * | proto: Split out DataCmdChecker into multiple types. | Gabriela Moldovan | 2025-10-16 | 3 | -13/+51 | |
| | | | | | | | | | | | | | | | | | | | | | | | The incoming one will be used for the exit relay implementation too. Also, with this change, receiving `CONNECTED` on an incoming stream will result in a clearer error message. Previously, the check against receiving `CONNECTED` on an incoming stream was bundled with the double-CONNECTED check for client data streams, so in the incoming stream case, the error message was misleading ("Received CONNECTED twice on a stream."). | |||||
| | * | proto: Do not fully-qualify Arc. | Gabriela Moldovan | 2025-10-16 | 1 | -3/+3 | |
| | | | | | | | | | | | `Arc` is already in scope, and not fully-qualifying it makes it more readable. | |||||
| | * | proto: Replace fully-qualified paths with use (fmt). | Gabriela Moldovan | 2025-10-16 | 2 | -8/+2 | |
| | | | ||||||
| | * | proto: Replace fully-qualified paths with use. | Gabriela Moldovan | 2025-10-16 | 2 | -8/+8 | |
| | | | ||||||
| | * | proto: Reformat imports post-move. | Gabriela Moldovan | 2025-10-16 | 10 | -12/+11 | |
| | | | ||||||
| | * | proto: Avoid crate-pub reexports of shared types from the client module. | Gabriela Moldovan | 2025-10-16 | 11 | -20/+19 | |
| | | | | | | | | | | | This makes it easier to see which parts are implementation-agnostic (i.e. do not import from crate::client). | |||||
| | * | proto: Move cmdchecker under top-level stream module. | Gabriela Moldovan | 2025-10-16 | 3 | -2/+2 | |
| | | | | | | | | | | | The CmdChecker will be used by relays too, so I am moving it to the shared `stream` module. | |||||
| * | | Merge branch 'fix-watch-single-file-flake' into 'main' | wesleyac | 2025-10-16 | 1 | -3/+5 | |
| |\ \ | | | | | | | | | | | | | | | | | | | arti config: Fix flaky watch_single_file test. Closes #1607 See merge request tpo/core/arti!2503 | |||||
| | * | | arti config: Fix flaky watch_single_file test in a more principled way. | Wesley Aptekar-Cassels | 2025-10-07 | 1 | -12/+4 | |
| | | | | | | | | | | | | | Thanks to gabi for the suggestion. | |||||
| | * | | arti config: Fix flaky watch_single_file test. | Wesley Aptekar-Cassels | 2025-10-07 | 1 | -2/+12 | |
| | | | | | | | | | | | | | Fixes: #1607 | |||||
| * | | | Fix new clippy nightly warning about subtracting Durations. | Nick Mathewson | 2025-10-16 | 2 | -8/+14 | |
| | | | | ||||||
| * | | | Remove a couple of unused imports. | Nick Mathewson | 2025-10-16 | 2 | -2/+1 | |
| | |/ |/| | ||||||
| * | | Merge branch 'outbound-relay-doc-fix' into 'main' | Nick Mathewson | 2025-10-16 | 1 | -1/+1 | |
| |\ \ | | | | | | | | | | | | | proto: Fix typo in OutboundRelayLayer docs. See merge request tpo/core/arti!3346 | |||||
| | * | | proto: Fix typo in OutboundRelayLayer docs. | Gabriela Moldovan | 2025-10-08 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | `OutboundRelayLayer::decrypt_outbound()` is for decrypting cells moving *away* from the client (in the "forward direction"). | |||||
| * | | | Merge branch 'parse2-routerstatus' into 'main' | Ian Jackson | 2025-10-15 | 19 | -118/+823 | |
| |\ \ \ | | | | | | | | | | | | | | | | | tor-netdoc: Be able to parse routerstatus entries in votes See merge request tpo/core/arti!3351 | |||||
| | * | | | tor-netdoc: Add a lot of spec cross-references and tidy docs | Ian Jackson | 2025-10-15 | 3 | -15/+101 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add links to torspec, and other cross-references. Change heading fof `IgnoredPublicationTimeSp` to `publication` to match spec and distinguish it from the `published` item in netstatus docs. Align several descriptions with the wording from torspec. Remove a restatement of a fact from the spec, on the `valid_until` field in `Lifetime`. | |||||
| | * | | | tor-netdoc: Typo fixes in docs | Ian Jackson | 2025-10-15 | 2 | -2/+2 | |
| | | | | | ||||||
| | * | | | tor-netdoc: tests: Print out vote routerstatus | Ian Jackson | 2025-10-15 | 1 | -1/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I have eyeballed the output and it seems to look right. Eventually we'll have encoing code and then we'll be able to check that things round-trip. | |||||
| | * | | | tor-netdoc: tests: Move vote parsing test to netstatus.rs | Ian Jackson | 2025-10-15 | 2 | -12/+21 | |
| | | | | | | | | | | | | | | | | | | | | | This is a test of real document structs now, albeit that it still uses one of the types from `poc`. | |||||
| | * | | | tor-netdoc: tests: Use real `RouterStatus` in `parse2::poc` | Ian Jackson | 2025-10-15 | 2 | -9/+8 | |
| | | | | | | | | | | | | | | | | | This now exercises all the new parsing code. | |||||
| | * | | | tor-netdoc: impl parse2 for RouterStatus entry | Ian Jackson | 2025-10-15 | 2 | -1/+85 | |
| | | | | | ||||||
| | * | | | tor-netdoc: Type for parsing `m` lines in votes | Ian Jackson | 2025-10-15 | 2 | -1/+39 | |
| | | | | | ||||||
| | * | | | tor-netdoc: Add ConsensusMethod type | Ian Jackson | 2025-10-15 | 1 | -3/+46 | |
| | | | | | ||||||
| | * | | | tor-netdoc: Add types for hashes with their accompanying alg id | Ian Jackson | 2025-10-15 | 2 | -3/+143 | |
| | | | | | ||||||
| | * | | | tor-netdoc: B64: impl Clone, [ConstantTime]Eq, Display, Debug | Ian Jackson | 2025-10-15 | 1 | -0/+27 | |
| | | | | | ||||||
| | * | | | tor-netdoc: B64 helper type: Expose | Ian Jackson | 2025-10-15 | 3 | -4/+6 | |
| | | | | | ||||||
| | * | | | tor-netdoc: impl parse2 multiplicity for BTreeSet | Ian Jackson | 2025-10-15 | 3 | -3/+36 | |
| | | | | | ||||||
| | * | | | tor-netdoc: RouterStatus: Add `publication` argument field | Ian Jackson | 2025-10-15 | 6 | -3/+28 | |
| | | | | | ||||||
| | * | | | tor-netdoc: Mirror netdoc structure in RouterStatus document digests | Ian Jackson | 2025-10-15 | 5 | -7/+38 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In md consensuses the referenced document digest is in the `m` field. This seems to be to make it similar to votes. But really the function of the md desc hash in md rs entries is much more parallel to the plain desc hash in plain rs entries. The way the spec has done it means the `r` item has a needlessly different syntax. | |||||
| | * | | | tor-netdoc: Provide doc_digest on vote routerstatus entries | Ian Jackson | 2025-10-15 | 2 | -6/+12 | |
| | | | | | ||||||
| | * | | | tor-netdoc: RouterStatus: Split out the `r` item's fields | Ian Jackson | 2025-10-15 | 5 | -28/+62 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes the Rust structure closer to the document structure. Arguably, this makes the Rust structure worse. But: I'm really very keen on avoiding handwritten parsing and printing code. And, in the longer term, having this linkage will hopefully apply pressure to avoid further protocol complexity/variation/chaos. | |||||
| | * | | | tor-netdoc: impl parse2 for rs::Version | Ian Jackson | 2025-10-15 | 1 | -0/+10 | |
| | | | | | ||||||
| | * | | | tor-netdoc: impl parse2 for RelayFlags | Ian Jackson | 2025-10-15 | 1 | -0/+13 | |
| | | | | | ||||||
| | * | | | tor-netdoc: split up RelayFlags parsing (fmt) | Ian Jackson | 2025-10-15 | 1 | -5/+1 | |
| | | | | | ||||||
| | * | | | tor-netdoc: split up RelayFlags parsing | Ian Jackson | 2025-10-15 | 1 | -14/+47 | |
| | | | | | ||||||
| | * | | | tor-netdoc: impl parse2 traits for RelayWeight | Ian Jackson | 2025-10-15 | 1 | -0/+25 | |
| | | | | | ||||||
| | * | | | tor-netdoc: split up RelayWeight parsing (fmt) | Ian Jackson | 2025-10-15 | 1 | -4/+2 | |
| | | | | | ||||||
| | * | | | tor-netdoc: split up RelayWeight parsing | Ian Jackson | 2025-10-15 | 1 | -2/+9 | |
| | | | | | | | | | | | | | | | | | We're going to call from_net_params from the parse2 trait impl. | |||||
| | * | | | tor-netdoc: impl parse2 traits for protovers | Ian Jackson | 2025-10-15 | 3 | -1/+25 | |
| | | | | | ||||||
| | * | | | tor-netdoc: Provide convenience method for handling invalid arguments | Ian Jackson | 2025-10-15 | 2 | -0/+15 | |
| | | | | | ||||||
| | * | | | tor-netdoc: Introduce ArgumentNotPresent placeholder utility type | Ian Jackson | 2025-10-15 | 2 | -3/+22 | |
| | | | | | ||||||
| | * | | | tor-netdoc: Make Ignored part of a module | Ian Jackson | 2025-10-15 | 1 | -16/+23 | |
| | | | | | | | | | | | | | | | | | | | | | The rest of types::misc does this. We should probably follow suit for consistency. Also we're going to want to add soemthing here. | |||||
| | * | | | tor-netdoc: impl Default for Ignored | Ian Jackson | 2025-10-15 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | This omission seems like an oversight. I thought I wanted it, but it turns out I don't - but I think it should still be there. | |||||
| | * | | | tor-netdoc: parse2: export NetdocSigned trait | Ian Jackson | 2025-10-15 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | This seems to have been overlooked. | |||||
| * | | | | Merge branch 'flow-ctrl-limits' into 'main' | David Goulet | 2025-10-15 | 4 | -82/+78 | |
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | tor-proto: Relax XON/XOFF receive limits See merge request tpo/core/arti!3360 | |||||
| | * | | | | tor-proto: update unit test | Steven Engler | 2025-10-14 | 1 | -27/+24 | |
| | | | | | | ||||||
| | * | | | | tor-proto: relax XON limits | Steven Engler | 2025-10-14 | 1 | -3/+13 | |
| | | | | | | ||||||
