summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | Merge branch 'log-unexpected-params' into 'main'Jim Newsome2025-06-231-2/+75
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | Add logging for possibly-bad net params See merge request tpo/core/arti!3043
| * | | | | NetParameters::saturating_update: log when a value is out of rangeJim Newsome2025-06-231-1/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On one hand, this can be quite noisy when something *is* out of range. On the other hand, a clamped value shouldn't normally happen, and is likely to result in surprising and otherwise difficult-to-debug behavior.
| * | | | | Log unrecognized net paramsJim Newsome2025-06-231-1/+4
| | | | | |
* | | | | | Merge branch 'rtcompat-spawn-block-detach' into 'main'Ian Jackson2025-06-231-0/+9
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | tor-rtmock: Blocking::spawn_blocking: clarify drop behaviour See merge request tpo/core/arti!3062
| * | | | | tor-rtmock: Blocking::blocking_io: add clarificationIan Jackson2025-06-231-0/+3
| | | | | |
| * | | | | tor-rtmock: Blocking::spawn_blocking: clarify drop behaviourIan Jackson2025-06-231-0/+6
|/ / / / / | | | | | | | | | | | | | | | | | | | | Explain that you don't need to .await the returned future, and say what happens when you drop it.
* | | | | Merge branch 'the-dependency-formerly-known-as-once-cell' into 'main'Nick Mathewson2025-06-2340-115/+96
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | various crates: Removed dependency on `once_cell` See merge request tpo/core/arti!3051
| * | | | | tor-rpcbase: Removed dependency on `once_cell`hashcatHitman2025-06-146-14/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::sync::Lazy` with `std::sync::LazyLock`. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | tor-relay-selection: Removed dependency on `once_cell`hashcatHitman2025-06-143-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::sync::Lazy` with `std::sync::LazyLock`. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | tor-netdoc: Removed dependency on `once_cell`hashcatHitman2025-06-1411-42/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::sync::Lazy` with `std::sync::LazyLock`. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | tor-log-ratelim: Removed dependency on `once_cell`hashcatHitman2025-06-144-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::sync::Lazy` with `std::sync::LazyLock`. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | tor-llcrypto: Removed dependency on `once_cell`hashcatHitman2025-06-143-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::sync::Lazy` with `std::sync::LazyLock`. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | tor-dirmgr: Removed dependency on `once_cell`hashcatHitman2025-06-144-8/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::sync::Lazy` with `std::sync::LazyLock`. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | tor-config-path: Removed dependency on `once_cell`hashcatHitman2025-06-144-10/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::sync::Lazy` with `std::sync::LazyLock`. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | tor-config: Removed dependency on `once_cell`hashcatHitman2025-06-143-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::sync::Lazy` with `std::sync::LazyLock`. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | tor-circmgr: Reduced dependency on `once_cell`hashcatHitman2025-06-141-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::sync::Lazy` with `std::sync::LazyLock`. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | fs-mistrust: Removed dependency on `once_cell`hashcatHitman2025-06-144-9/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::sync::Lazy` with `std::sync::LazyLock`. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | arti-relay: Removed dependency on `once_cell`hashcatHitman2025-06-144-9/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::sync::Lazy` with `std::sync::LazyLock`. Signed-off-by: hashcatHitman <[email protected]>
| * | | | | tor-error: Removed dependency on `once_cell`hashcatHitman2025-06-133-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced `once_cell::unsync::Lazy` with `std::cell::LazyCell`. Signed-off-by: hashcatHitman <[email protected]>
* | | | | | Merge branch '2043-fix-keys-desc' into 'main'gabi-2502025-06-232-3/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | arti: keys: Fix arti keys help message Closes #2043 See merge request tpo/core/arti!3060
| * | | | | | arti: keys: Fix arti keys help messagehjrgrn2025-06-232-3/+3
|/ / / / / /
* | | | | | Merge branch 'log-stderr' into 'main'Jim Newsome2025-06-176-21/+59
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | arti console logging: use stderr instead of stdout Closes #2024 See merge request tpo/core/arti!3020
| * | | | | arti console logging: use stderr instead of stdoutJim Newsome2025-06-173-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | Fixes #2024
| * | | | | shadow CI: have arti processes log to file instead of consoleJim Newsome2025-06-173-17/+54
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is in preparation to switch arti's console logging to use stderr instead of stdout. The test script currently fails the test if there are non-empty stderr files, since that's where rust panics end up getting recorded. It will also make it easier to see if there is any other non-logging output. Also disable trace-level logging for the tor_proto module. It's unclear exactly when/why this was added, but it's probably overkill other than when debugging.
* | | | | Merge branch 'dircache-proposal' into 'main'Ian Jackson2025-06-171-0/+564
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | doc: Document the WIP dircache storage model See merge request tpo/core/arti!3017
| * | | | | doc: Document the WIP dircache storage modelClara Engler2025-06-171-0/+564
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit documents the WIP storage model for the dircache. It is not finalized and several parts need discussion. See !3004
* | | | | Merge branch 'flow-ctrl' into 'main'opara2025-06-166-21/+322
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tor-proto: Add `DynamicRateLimitedWriter` See merge request tpo/core/arti!3033
| * | | | | tor-proto: use `DynamicRateLimitedWriter` in `DataWriter`Steven Engler2025-06-161-6/+17
| | | | | | | | | | | | | | | | | | | | | | | | This uses just a placeholder `Empty` stream for config updates.
| * | | | | tor-proto: improve `TokioAsyncWrite` compat implSteven Engler2025-06-161-10/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We want the tokio trait to call into the futures trait, rather than having each trait duplicate the logic of calling into the inner writer. This is less error-prone.
| * | | | | tor-proto: add `DynamicRateLimitedWriter`Steven Engler2025-06-164-2/+296
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a `Writer` rate limiter which can receive rate limit updates from a `Stream`.
| * | | | | tor-proto: make `visibility` dependency always availableSteven Engler2025-06-161-3/+3
|/ / / / / | | | | | | | | | | | | | | | `RateLimitedWriter` will use this in a future commit.
* | | | | Merge branch 'abolish-leg-id' into 'main'David Goulet2025-06-169-172/+144
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-proto: Replace LegId/LegIdKey with UniqId. Closes #1999 See merge request tpo/core/arti!3053
| * | | | | tor-proto: Get rid of unnecessary extra variable.Gabriela Moldovan2025-06-161-3/+1
| | | | | |
| * | | | | tor-proto: Adapt the circ mutable state to removal of LegId.Gabriela Moldovan2025-06-161-9/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Circuits only have one identifier now, so we can remove the second, now-redundant ID.
| * | | | | tor-proto: Replace LegId/LegIdKey with UniqId.Gabriela Moldovan2025-06-138-166/+138
| | | | | | | | | | | | | | | | | | | | | | | | Closes #1999
| * | | | | tor-proto: Derive HasMemoryCost for UniqId.Gabriela Moldovan2025-06-131-1/+5
| | | | | |
* | | | | | Merge branch 'more-tunnel-id' into 'main'David Goulet2025-06-163-10/+29
|\| | | | | | |/ / / / |/| | | | | | | | | | | | | | tor-proto: Fix up the circuit's tunnel ID after inserting into conflux set. See merge request tpo/core/arti!3050
| * | | | tor-proto: Update doc links post-renaming.Gabriela Moldovan2025-06-131-1/+1
| | | | |
| * | | | tor-proto: Group conflux imports for consistency.Gabriela Moldovan2025-06-131-2/+1
| | | | |
| * | | | tor-proto: Rename function to clarify transition to a new tunnel (fmt).Gabriela Moldovan2025-06-132-2/+9
| | | | |
| * | | | tor-proto: Rename function to clarify transition to a new tunnel.Gabriela Moldovan2025-06-132-3/+4
| | | | |
| * | | | tor-proto: Fix up the circuit's tunnel ID after inserting into conflux set.Gabriela Moldovan2025-06-132-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When adding new circuits to an existing tunnel, we need to fixup the internal TunnelId of those circuits (otherwise they will have the TunnelId of the old single-path "tunnel" reactor they were extracted from using `CtrlCmd::ShutdownAndReturnCircuit`). Spotted while writing some tests.
| * | | | tor-proto: Give ConfluxSet a TunnelId for logging purposes.Gabriela Moldovan2025-06-132-4/+14
|/ / / /
* | | | Merge branch 'tunnel-ids' into 'main'gabi-2502025-06-1210-86/+164
|\ \ \ \ | |/ / / |/| | | | | | | | | | | tor-proto: Assign tunnels unique IDs, update logs to include these IDs See merge request tpo/core/arti!3048
| * | | tor-proto: Use TunnelScopedCircId instead of UniqId for logging.Gabriela Moldovan2025-06-124-10/+12
| | | |
| * | | tor-proto: Add an identifier for a circuit within a tunnel.Gabriela Moldovan2025-06-121-0/+27
| | | | | | | | | | | | | | | | | | | | | | | | This type will help produce better logs (logging just the circuit ID would make it impossible to correlate said circuit with the tunnel it belongs to).
| * | | tor-proto: Assign a separate identifier to the tunnel reactor.Gabriela Moldovan2025-06-122-20/+17
| | | | | | | | | | | | | | | | | | | | | | | | The plan is to reuse this identifier for the future tunnel reactor updates channel (a channel for sending tunnel status updates to a central consumer).
| * | | tor-proto: Log channel, circuit, stream identifiers as structured fields.Gabriela Moldovan2025-06-129-72/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changes the `tor-proto` logs to not be prefixed with a channel/circuit/stream ID, but rather to have these IDs attached to the log as structured fields. This change is in preparation for the switch to using `TunnelId`s in the tunnel reactor instead of circuit `UniqId`s. The reason for the change to use structured fields is because future logs will likely need to log the `UniqId`s of the circuits in a tunnel, which will need to either be formatted somehow in the logs, or logged as a structured field (the latter seems like the better option, hence this preparatory change). IMO we should favor structured fields over formatted strings in the logs in general, but that is a bigger project, so I am only doing a spot fix for now.
| * | | tor-proto: Add a new identifier type for tunnels.Gabriela Moldovan2025-06-121-0/+22
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, a tunnel is uniquely identified by the `UniqId` of the first circuit added to the tunnel. This works, but the double-meaning of the `UniqId` is bound to cause confusion in the future (because it blurs the distinction between tunnels and circuits). This introduces a new `TunnelId` type which will replace `UniqId` in the tunnel reactor.
* | | Merge branch 'gsoc-list' into 'main'gabi-2502025-06-1229-13/+616
|\ \ \ | | | | | | | | | | | | | | | | arti: Add keys list and keys list-keystores See merge request tpo/core/arti!3023