summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | Cleanup: Remove a variable renameNick Mathewson2025-10-201-4/+2
| | | |
| * | | Refactor: turn circ_extensions_from_settings into a methodNick Mathewson2025-10-203-63/+62
|/ / / | | | | | | | | | Closes #2067.
* | | Merge branch 'guardmgr-retry-logging' into 'main'wesleyac2025-10-162-8/+21
|\ \ \ | | | | | | | | | | | | | | | | Add more detail on retry times to GuardMgr logging See merge request tpo/core/arti!3153
| * | | tor-guardmgr: Ignore cognitive_complexity lint.Wesley Aptekar-Cassels2025-10-161-0/+1
| | | | | | | | | | | | | | | | | | | | While this is fairly nested, it's short, and most of the complexity is just in the logging code.
| * | | tor-guardmgr: Use humantime for some logging.Wesley Aptekar-Cassels2025-10-162-2/+2
| | | |
| * | | tor-guardmgr: Add retry time to guard failure warning.Wesley Aptekar-Cassels2025-10-161-7/+14
| | | | | | | | | | | | | | | | | | | | The "Next retry time unknown" message can't be hit right now, but could be if the code changes in the future.
| * | | tor-guardmgr: Include retry time in AllGuardsDown error message.Wesley Aptekar-Cassels2025-10-161-1/+6
| | | | | | | | | | | | | | | | | | | | I don't love shoving such a complicated expression in thiserror, but it seems nicer than writing the implementation by hand...
* | | | Merge branch 'ignore-watch-multiple-test' into 'main'gabi-2502025-10-161-0/+2
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | arti: Ignore reload_cfg::test::watch_multiple. See merge request tpo/core/arti!3364
| * | | arti: Ignore reload_cfg::test::watch_multiple.Wesley Aptekar-Cassels2025-10-161-0/+2
| |/ / | | | | | | | | | This is flaky, ignoring until #1607 is fixed.
* | | Merge branch 'parallelize-minimal-features-test' into 'main'wesleyac2025-10-162-2/+36
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | ci: Parallelize minimal-features-test Closes #2207 See merge request tpo/core/arti!3338
| * | | ci: Parallelize minimal-features-testWesley Aptekar-Cassels2025-10-162-2/+36
| |/ / | | | | | | | | | | | | | | | | | | 3 seems like a good number to start with, we can tune it later if needed. Fixes: #2207
* | | Merge branch 'cmd-checkers' into 'main'gabi-2502025-10-1614-64/+101
|\ \ \ | |/ / |/| | | | | | | | proto: Split out IncomingDataCmdChecker as a separate type See merge request tpo/core/arti!3348
| * | proto: Rename data cmd checkers to {Inbound,Outbound}DataCmdChecker (fmt)Gabriela Moldovan2025-10-161-1/+2
| | |
| * | proto: Rename data cmd checkers to {Inbound,Outbound}DataCmdCheckerGabriela Moldovan2025-10-167-19/+19
| | | | | | | | | | | | | | | | | | The previous "incoming" terminology was rather ambiguous. Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3348#note_3275337
| * | proto: Fix docs to refer to the right stream:: paths.Gabriela Moldovan2025-10-162-2/+2
| | |
| * | proto: Move IncomingDataCmdChecker to top-level stream module.Gabriela Moldovan2025-10-164-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 Moldovan2025-10-161-1/+1
| | |
| * | proto: Split out DataCmdChecker into multiple types.Gabriela Moldovan2025-10-163-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 Moldovan2025-10-161-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 Moldovan2025-10-162-8/+2
| | |
| * | proto: Replace fully-qualified paths with use.Gabriela Moldovan2025-10-162-8/+8
| | |
| * | proto: Reformat imports post-move.Gabriela Moldovan2025-10-1610-12/+11
| | |
| * | proto: Avoid crate-pub reexports of shared types from the client module.Gabriela Moldovan2025-10-1611-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 Moldovan2025-10-163-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'wesleyac2025-10-161-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-Cassels2025-10-071-12/+4
| | | | | | | | | | | | | | | | Thanks to gabi for the suggestion.
| * | | arti config: Fix flaky watch_single_file test.Wesley Aptekar-Cassels2025-10-071-2/+12
| | | | | | | | | | | | | | | | Fixes: #1607
* | | | Merge branch 'nightly_fixes' into 'main'Nick Mathewson2025-10-164-10/+15
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | Fix a couple of clippy warnings from nightly See merge request tpo/core/arti!3361
| * | | Fix new clippy nightly warning about subtracting Durations.Nick Mathewson2025-10-162-8/+14
| | | |
| * | | Remove a couple of unused imports.Nick Mathewson2025-10-162-2/+1
| | | |
* | | | Merge branch 'shellcheck' into 'main'gabi-2502025-10-161-1/+1
|\ \ \ \ | |/ / / |/| | | | | | | | | | | maint/reproducible_build: Use cp -R to placate shellcheck See merge request tpo/core/arti!3362
| * | | maint/reproducible_build: Use cp -R to placate shellcheckIan Jackson2025-10-161-1/+1
|/ / / | | | | | | | | | | | | | | | I'm not sure I agree with this complaint - it seems like foolish standards-wonkery - but GNU cp supports -R too and this change will fix CI on main.
* | | Merge branch 'outbound-relay-doc-fix' into 'main'Nick Mathewson2025-10-161-1/+1
|\ \ \ | | | | | | | | | | | | | | | | proto: Fix typo in OutboundRelayLayer docs. See merge request tpo/core/arti!3346
| * | | proto: Fix typo in OutboundRelayLayer docs.Gabriela Moldovan2025-10-081-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 Jackson2025-10-1519-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 docsIan Jackson2025-10-153-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 docsIan Jackson2025-10-152-2/+2
| | | | |
| * | | | tor-netdoc: tests: Print out vote routerstatusIan Jackson2025-10-151-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.rsIan Jackson2025-10-152-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 Jackson2025-10-152-9/+8
| | | | | | | | | | | | | | | | | | | | This now exercises all the new parsing code.
| * | | | tor-netdoc: impl parse2 for RouterStatus entryIan Jackson2025-10-152-1/+85
| | | | |
| * | | | tor-netdoc: Type for parsing `m` lines in votesIan Jackson2025-10-152-1/+39
| | | | |
| * | | | tor-netdoc: Add ConsensusMethod typeIan Jackson2025-10-151-3/+46
| | | | |
| * | | | tor-netdoc: Add types for hashes with their accompanying alg idIan Jackson2025-10-152-3/+143
| | | | |
| * | | | tor-netdoc: B64: impl Clone, [ConstantTime]Eq, Display, DebugIan Jackson2025-10-151-0/+27
| | | | |
| * | | | tor-netdoc: B64 helper type: ExposeIan Jackson2025-10-153-4/+6
| | | | |
| * | | | tor-netdoc: impl parse2 multiplicity for BTreeSetIan Jackson2025-10-153-3/+36
| | | | |
| * | | | tor-netdoc: RouterStatus: Add `publication` argument fieldIan Jackson2025-10-156-3/+28
| | | | |
| * | | | tor-netdoc: Mirror netdoc structure in RouterStatus document digestsIan Jackson2025-10-155-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 entriesIan Jackson2025-10-152-6/+12
| | | | |
generated by cgit v1.2.3 (git 2.46.0) at 2026-08-27 19:32:11 +0000