summaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge branch 'circ-react-report' into 'main'opara2025-08-188-47/+159
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-error,arti: Support tracing fields in the `_report!` macros Closes #2096 and #2116 See merge request tpo/core/arti!3142
| * | | | | tor-error: allow only error given to `_report!` macrosSteven Engler2025-08-181-0/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is useful when you create an `internal!` error and then immediately report it. If the `_report!` macro also requires a message, then you need to provide two messages for the error, which doesn't always make sense. This is already possible anyways with `warn_report!(e,)`. Now you don't need the comma.
| * | | | | misc: cleanup now that `_report!` macros support fieldsSteven Engler2025-08-184-7/+5
| | | | | |
| * | | | | tor-proto: report tunnel/channel id as a fieldSteven Engler2025-08-182-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | This restores the pre-374889d34aa0 behaviour.
| * | | | | arti: log errors using `tor_error::ErrorReport`Steven Engler2025-08-181-1/+14
| | | | | |
| * | | | | arti: log error fields lastSteven Engler2025-08-182-0/+99
| | | | | |
| * | | | | tor-error: update `_report!` macros to support tracing fieldsSteven Engler2025-08-181-38/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This requires that we put the error in a field. There are two issues that we'll fix in the next two commits: 1. Errors are no longer formatted with `ErrorReport`. 2. Errors will be in the first field, but long (or multiline) error messages will cause other fields to be pushed to the end where they're harder to find.
* | | | | | Merge branch 'document-max-mem-overhead' into 'main'Nick Mathewson2025-08-181-4/+8
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | Note that not all memory is tracked by memqouta See merge request tpo/core/arti!3165
| * | | | | Add a disclaimer that not all memory usage is trackedNick Mathewson2025-08-181-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | Partially addresses #2102.
| * | | | | Light reformatting on system.memory documentation.Nick Mathewson2025-08-181-4/+5
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | This mainly separates the two options, and adds the word "tracked".
* | | | | Merge branch 'relay-reactor-placeholder' into 'main'David Goulet2025-08-1854-163/+494
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | proto: Add a placeholder for the relay reactor. See merge request tpo/core/arti!3162
| * | | | | proto: Reformat a paragraph in the README.Gabriela Moldovan2025-08-181-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | Wraps an overly-long line for consistency.
| * | | | | proto: Update README with new DataStream path.Gabriela Moldovan2025-08-181-1/+1
| | | | | |
| * | | | | proto: Fix tests post-crate reorg (fmt).Gabriela Moldovan2025-08-182-2/+2
| | | | | |
| * | | | | proto: Fix tests post-crate reorg.Gabriela Moldovan2025-08-182-2/+2
| | | | | |
| * | | | | proto: Avoid calling the relay reactor a tunnel reactor.Gabriela Moldovan2025-08-181-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A "tunnel" is a higher level concept we'll want to avoid using from now on when talking about the proto implementation.
| * | | | | proto: Move the `stream` module under `client` (fmt).Gabriela Moldovan2025-08-189-21/+23
| | | | | |
| * | | | | proto: Move the `stream` module under `client` (breaking).Gabriela Moldovan2025-08-1833-58/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `stream` module is client-specific, for the most part, so I am moving it under `client`. Later on, we will factor out the parts that can be shared with the relay implementation. Note: this is a breaking change as the deleted `stream` module was `pub`. We could've kept the module and reexported from it the public types from `tor_proto::client::stream`, but I think it's better to have this `client` namespacing, because it makes the separation between the client and relay parts clearer.
| * | | | | proto: Rename the `tunnel` module to `client` (fmt).Gabriela Moldovan2025-08-1816-46/+46
| | | | | |
| * | | | | proto: Rename the `tunnel` module to `client`.Gabriela Moldovan2025-08-1836-96/+95
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The implementation from `tunnel` is client-specific, so we are renaming the module accordingly. The more generic parts will be pulled into a separate module in a future commit.
| * | | | | proto: Reorganize relay_tunnel module.Gabriela Moldovan2025-08-184-15/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reorganizes the `relay_tunnel` module as per @dgoulet's [suggestion]. [suggestion]: https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3162/diffs#note_3240092
| * | | | | proto: Add a placeholder for the relay reactor.Gabriela Moldovan2025-08-181-0/+231
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a placeholder, and will likely change quite a bit in the near future. In particular, much of this is copied from the client tunnel reactor (a future change will refactor both of them to reduce/minimize code duplication). I'm adding this placeholder because the channel code will soon need the ability to create and launch circuit reactors.
| * | | | | proto: Make unwrap_or_shutdown pub(crate).Gabriela Moldovan2025-08-181-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | We will soon need this in the relay reactor.
| * | | | | proto: Add a ChannelProvider trait.Gabriela Moldovan2025-08-183-0/+74
| | | | | | | | | | | | | | | | | | | | | | | | Part of #1447
| * | | | | proto: Add a new relay_tunnel module.Gabriela Moldovan2025-08-182-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new relay tunnel reactor will live in this module for now. This is temporary, as I expect we will soon need to reorganize this crate a little bit, to more clearly separate the client-specific parts from the relay ones.
| * | | | | proto: Add an experimental relay feature.Gabriela Moldovan2025-08-151-0/+2
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | The relay-specific logic, including the new relay tunnel/circuit reactor, will be gated behind this feature.
* | | | | Merge branch 'fix-nightly-warn' into 'main'Nick Mathewson2025-08-1811-12/+5
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | Fix clippy errors on nightly See merge request tpo/core/arti!3148
| * | | | clippy: fix `clippy::duplicated_attributes` warningsSteven Engler2025-08-117-8/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ```text warning: duplicated attribute --> crates/tor-hsservice/src/timeout_track.rs:630:14 | 630 | #![allow(clippy::needless_pass_by_value)] // TODO hoist into standard lint block | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ```
| * | | | clippy: fix `clippy::implicit_clone` errorsSteven Engler2025-08-114-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ```text error: implicitly cloning a `String` by calling `to_string` on its dereferenced type --> crates/tor-config/src/lib.rs:109:32 | 109 | V::String(_, s) => s.to_string(), | ^^^^^^^^^^^^^ help: consider using: `s.clone()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#implicit_clone note: the lint level is defined here --> crates/tor-config/src/lib.rs:20:9 | 20 | #![deny(clippy::implicit_clone)] | ^^^^^^^^^^^^^^^^^^^^^^ ```
* | | | | tor-netdoc: Run rustfmt againIan Jackson2025-08-182-2/+2
| | | | | | | | | | | | | | | | | | | | Not sure how this is needed now, after rebase, but whatever.
* | | | | tor-netdoc: Downgrade two TODOs re NsSomething -> PlainSomethingIan Jackson2025-08-181-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | Let's not entangle this MR with such an incompatible change. I will do this as a followup.
* | | | | tor-netdoc: varieties: Rename per_variety to each_varietyIan Jackson2025-08-183-28/+26
| | | | |
* | | | | tor-netdoc: varieties: Rename macro definition moduleIan Jackson2025-08-183-6/+5
| | | | |
* | | | | tor-netdoc: Rename "species" to "variety"Ian Jackson2025-08-186-63/+62
| | | | |
* | | | | tor-netdoc: varieties: Fix typo on doc commentIan Jackson2025-08-181-1/+1
| | | | |
* | | | | tor-netdoc: varieties: Fix ns_choose for plainIan Jackson2025-08-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3139#note_3239863
* | | | | tor-netdoc: rs: Implement orport_addrs in terms of addrsIan Jackson2025-08-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | It's not clear why this method exists, but I'm not tidying this up now.
* | | | | tor-netdoc: Add commentary about namespacingIan Jackson2025-08-183-0/+25
| | | | |
* | | | | tor-netdoc: Add a TODO comment about possibly changing the structureIan Jackson2025-08-181-0/+3
| | | | |
* | | | | tor-netdoc: Add comment to ns_export_per_speciesIan Jackson2025-08-182-0/+2
| | | | | | | | | | | | | | | | | | | | And add a blocking todo about renaming it.
* | | | | tor-netdoc: Reference parse2/multiplicity.rs.Ian Jackson2025-08-181-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | (This isn't in this branch but it will be if we rebase, and also it will be present when we merge.)
* | | | | tor-netdoc: species/variety: Fix typoIan Jackson2025-08-181-1/+1
| | | | |
* | | | | tor-netdoc: species/variety: Say to rename the macrology fileIan Jackson2025-08-181-0/+1
| | | | |
* | | | | tor-netdoc: species/variety: Fix a reference to "this module"Ian Jackson2025-08-181-1/+1
| | | | |
* | | | | tor-netdoc: species/variety: Add a big overview textIan Jackson2025-08-181-0/+51
| | | | | | | | | | | | | | | | | | | | I've started using the new "variety" terminology here.
* | | | | tor-netdoc: species/variety: Add some cfg's to avoid unused import warningsIan Jackson2025-08-181-0/+3
| | | | |
* | | | | tor-netdoc: species/variety: Properly add blocking TODOsIan Jackson2025-08-181-1/+2
| | | | |
* | | | | Revert "tor-netdoc: Link to the generated macro, for one of them (poc)"Ian Jackson2025-08-184-20/+3
| | | | | | | | | | | | | | | | | | | | This reverts commit 084ad9a716da8e44616a54fdfbba0140f5a7704e.
* | | | | tor-netdoc: species: Improve per_species heading, clarificationIan Jackson2025-08-181-1/+1
| | | | |
* | | | | tor-netdoc: species: Improve per_species headingIan Jackson2025-08-181-1/+4
| | | | |