aboutsummaryrefslogtreecommitdiff
path: root/crates/arti
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'all_bind_addr_fail' into 'main'Jim Newsome10 days1-1/+6
|\ | | | | | | | | refactor: fail when all addresses bind fail See merge request tpo/core/arti!4309
| * bind_dns_resolver: fail when no addresses in a group can bindSteven Masnada10 days1-1/+6
| |
* | Apply 1 suggestion(s) to 1 file(s)Nick Mathewson10 days1-1/+1
| | | | | | Co-authored-by: gabi-250 <[email protected]>
* | Reindent impl block.Nick Mathewson2026-08-131-87/+88
| |
* | Move reload_configuration method.Nick Mathewson2026-08-131-40/+37
| | | | | | | | Code movement only.
* | reload_cfg: Refactor to use a CfgMgr object.Nick Mathewson2026-08-132-87/+134
|/ | | | | | | | | Right now this just helps us keep the parts of the configuration-reloading logic in one place, and clarifies what needs to be owned by the watcher thread and what doesn't. Moving forward, this will help make the configuration something that RPC can inspect and change.
* Merge branch 'dns_qdcount_must_be_one' into 'main'Nick Mathewson2026-08-121-68/+71
|\ | | | | | | | | fix: the number of dns queries should be 1 See merge request tpo/core/arti!4281
| * fix: the number of dns queries should be 1steven2026-08-041-68/+71
| |
* | arti: Fix compilation of http-connect without rpcNick Mathewson2026-08-101-1/+1
|/ | | | Closes #2653.
* Bump version for arti crateSteven Engler2026-08-031-1/+1
| | | | | | ```bash cargo set-version --bump patch -p arti ```
* Bump versions for tor-* and arti-* cratesSteven Engler2026-08-031-26/+26
| | | | | | | ```bash readarray -t bump < <(maint/list-crates | grep -P '^tor-|^arti-') for crate in "${bump[@]}"; do cargo set-version --bump minor -p "${crate}"; done ```
* Merge branch 'testdata-export' into 'main'Ian Jackson2026-07-271-1/+1
|\ | | | | | | | | tor-netdoc testdata-live: Export for the benefit of other crates See merge request tpo/core/arti!4229
| * Require derive-deftly 1.11.4Ian Jackson2026-07-231-1/+1
| | | | | | | | | | | | | | | | | | We're about to *use* the fix that was in 1.11.4. We already updated the lockfile in !4228, so this is just Cargo.toml changes (which will prevent the minimal-versions tests failing when we change the code to rely on the fix). git-grep -l '^derive-deftly' |xargs perl -i~ -pe 's{\b\Q1.11.3\E\b}{1.11.4}g'
* | arti: Ratelimit proxy error reportssyphyr2026-07-232-3/+47
|/ | | | Implement log_ratelim! in report_proxy_error
* Use "extend" for arti ResponseBuilder extension traitIan Jackson2026-07-162-13/+4
|
* Merge branch 'remove-lint' into 'main'Jim Newsome2026-07-157-11/+2
|\ | | | | | | | | | | | | Removed unnecessary lint Closes #2556 See merge request tpo/core/arti!4210
| * Remove now-unneeded allow(clippy::cognitive_complexity)Jim Newsome2026-07-156-9/+0
| |
| * add_warning: add reference to arti#2556Jim Newsome2026-07-152-2/+2
| |
| * Removed unnecessary lintpryty262026-07-152-2/+2
| | | | | | | | Removed unnecessary lint
* | arti: subcommands: keys: Fix clippy::for-kv-maphjrgrn2026-07-131-2/+2
| |
* | multiple crates: Fix clippy warningshjrgrn2026-07-103-7/+7
|/
* Cargo plumbing to expose HS extension negotiationNick Mathewson2026-07-021-0/+5
|
* Upgrade to itertools 0.15.0Nick Mathewson2026-07-011-2/+2
| | | | | | Additionally, fix itertools usage in maybenot_padding.rs The definition of `Position` changed in 0.15.0.
* Version bumps for 2.5.0Nick Mathewson2026-06-301-32/+32
| | | | | | | | | | | | | Closes #2617. We've lucked out this time, and it turns out that every one of our published crates gets a minor bump. So this was generated with: ``` for cr in $(./maint/list-crates); do cargo set-version -p $cr --bump minor done ```
* arti: remove references to old `trace_filter` in commentsSteven Engler2026-06-252-2/+2
|
* arti: subcommands: keys: Fix compact outputhjrgrn2026-06-221-1/+1
|
* Merge branch 'keys-list-tests' into 'main'opara2026-06-1811-597/+33
|\ | | | | | | | | arti: Rewrite hand-written tests using `trycmd` See merge request tpo/core/arti!4126
| * arti: Gate the "arti keys" tests behind onion-service-serviceGabriela Moldovan2026-06-171-2/+6
| | | | | | | | | | | | | | | | Now that these test more than just the help output, we need to gate them behind the `onion-service-service` feature: the tests involve some C Tor keystores, and require at least one onion service to be configured, because onion services (and clients) are the only thing these keystores are used for.
| * arti: Remove no longer used test helperGabriela Moldovan2026-06-171-13/+1
| |
| * arti: Remove now-obsolete keys test moduleGabriela Moldovan2026-06-173-581/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We now have `trycmd` tests that cover most cases, so I am removing this module. The reason this was introduced in the first place is in the module level docs ```rust //! The `assert_cmd` crate is used here instead of the preferred `trycmd` (see //! [`README`](../README.md)) because the output of `keys list` is not deterministic across //! different machines. The design choices of some components are workarounds for this limitation. ``` But this is no longer an issue now that the `keys list` output is sorted. Ideally, all tests would be written this way (we have a few more modules in `cli_tests` that we should probably port to `trycmd` sometime). Note: some of the tests I'm removing here don't have a `trycmd` counterpart in `arti/tests/testcases/keys`. In my opinion, this is perfectly fine, because the old tests were unnecessarily testing all the possible configurations involving 1-2 keystores (one arti, and one C Tor): it is not the responsibility of the CLI tests to cover all these permutations (they should be tested in `tor-keymgr`).
| * arti: Add a trycmd-based test for arti keys list error handlingGabriela Moldovan2026-06-173-0/+5
| | | | | | | | | | This is a `trycmd`-based replacement for the `list_unregistered_keystore_fails()` test, which will be removed soon.
| * doc/keys.md: Un-ignore some of the example snippetsGabriela Moldovan2026-06-172-1/+16
| | | | | | | | | | | | | | | | | | This un-ignores some of the output snippets from the `doc/keys.md` file, so that our `trycmd` test runner will actually run them. This will force us to keep the snippet in sync with the code. This involved adding some extra config for the test C Tor keystore, and tweaking the output to match what the commands actually print out.
| * arti: Sort the arti keys list outputGabriela Moldovan2026-06-171-0/+5
| | | | | | | | | | This will make the output deterministic and enable us to write some `trycmd` tests in the form of markdown output snippets.
* | Merge branch 'msrv-1.91' into 'main'gabi-2502026-06-181-1/+1
|\ \ | | | | | | | | | | | | Bump MSRV to 1.91 See merge request tpo/core/arti!4105
| * | Bump MSRV to 1.91Clara Engler2026-06-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit bumps the MSRV to 1.91 which was released on 2025-10-30. The Cargo.toml files were updated as follows: ```sh git ls-files | \ grep ".*Cargo\.toml$" | \ xargs sed -i '' 's/^rust-version = "1\.89"$/rust-version = "1\.91"/g' ``` The following files were updated manually: ``` modified: .gitlab-ci.yml modified: README.md modified: flake.nix modified: maint/docker-android/Dockerfile ```
* | | Merge branch 'misc_string_slices_1' into 'main'Clara Engler2026-06-181-2/+1
|\ \ \ | |_|/ |/| | | | | | | | Fix string_slice exceptions outside of netdoc See merge request tpo/core/arti!4092
| * | arti: remove string slicesNick Mathewson2026-06-101-2/+1
| | |
* | | arti, keymgr: s/IDEA/TODOGabriela Moldovan2026-06-171-1/+1
| | | | | | | | | | | | | | | | | | It's better to use "TODO" like we do in the rest of Arti, because otherwise this has a chance of being forgotten (because nobody really greps for "IDEA").
* | | arti: Don't mention entry validity in the help messageGabriela Moldovan2026-06-172-2/+2
| | | | | | | | | | | | | | | | | | This actually applies to unrecognized entries too, and besides, most users aren't going to think in terms of "valid" vs "invalid" entries, so it's best to omit this qualifier.
* | | arti: subcommands: keys: Rework output of keys listhjrgrn2026-06-174-149/+384
| |/ |/|
* | Merge branch 'cgo-stable' into 'main'Nick Mathewson2026-06-101-2/+2
|\ \ | | | | | | | | | | | | | | | | | | Mark "counter-galois-onion" as stable Closes #2550 See merge request tpo/core/arti!4069
| * | Mark "counter-galois-onion" as stableNick Mathewson2026-06-041-2/+2
| | |
* | | arti: add "dep:" prefix to 'rpc' feature dependenciesSteven Engler2026-06-101-1/+1
| |/ |/| | | | | | | | | This is technically a breaking change since it removes the tor-rpc-connect, tor-rpcbase, and arti-rpcserver features from arti. But I think these implicit features should not be expected to be stable.
* | Merge branch 'clippy-string-slice' into 'main'Nick Mathewson2026-06-1012-0/+12
|\ \ | | | | | | | | | | | | Lint for clippy::string_slice See merge request tpo/core/arti!4086
| * | everywhere: Add #[allow(clippy::string_slice)]Clara Engler2026-06-091-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | This commit adds #[allow(clippy::string_slice)] to all functions in the code where string slices are used, alongside a TODO comment. We do this add the function header to have it consistent, as things like expression based allow's are still experimental.
| * | maint: Run maint/add_warning to deny string slicesClara Engler2026-06-0911-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit executes maint/add_warning with the just added change to deny string slices except in tests. I recommend auditing this by checking out the previous commit followed by running the script yourself and then verifying that the diff is identical to this commit. This commit makes cargo clippy fail. We will add exceptions in the next commit.
* | | Merge branch 'cc-default' into 'main'Nick Mathewson2026-06-101-0/+1
|\ \ \ | | | | | | | | | | | | | | | | Add 'flowctl-cc' to 'default' See merge request tpo/core/arti!4042
| * | | arti: add 'flowctl-cc' to 'default'Steven Engler2026-06-011-0/+1
| | | |
* | | | Bump derive-deftly to 1.11.3Ian Jackson2026-06-091-1/+1
| |/ / |/| | | | | | | | New beta semver policy means we should pin the patchlevel.
* | | proto: Move RateLimitedWriter to tor-async-utilsDavid Goulet2026-06-081-1/+1
| | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>