summaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | proto: use assert_matches in circuit.rs tests.Nick Mathewson2025-11-042-5/+7
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unlike assert!(matches!(..)), assert_matches prints the value of the expression on a failure. Probably we should use this macro more widely in the future. This might help diagnose recurrences of #2232
* | | | | Merge branch 'use_futures_copy' into 'main'Nick Mathewson2025-11-036-187/+61
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | Replace copy_interactive with futures-copy. See merge request tpo/core/arti!3416
| * | | | Replace copy_interactive with futures-copy.Nick Mathewson2025-10-306-187/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change lets us avoid spawning extra tasks (due to one-direction nature of copy_interactive), and avoid some lock contention (due to use of AsyncReadExt::split). Addresses part of #786.
* | | | | arti: fix deprecation warnings in assert_cmd libSteven Engler2025-10-314-8/+9
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ```text warning: use of deprecated associated function `assert_cmd::Command::cargo_bin`: incompatible with a custom cargo build-dir, see instead `cargo::cargo_bin_cmd!` --> crates/arti/tests/cli_tests/hsc.rs:26:28 | 26 | let mut cmd = Command::cargo_bin("arti").unwrap(); | ^^^^^^^^^ | = note: `#[warn(deprecated)]` on by default ```
* | | | proto: Move a TODO to CircSyncViewGabriela Moldovan2025-10-302-3/+3
| | | |
* | | | proto: Replace ClientCircSyncView in IncomingStreamRequestFilterGabriela Moldovan2025-10-306-8/+11
| | | |
* | | | proto: Add a new CircSyncView typeGabriela Moldovan2025-10-302-0/+40
| | | | | | | | | | | | | | | | | | | | | | | | This will replace ClientCircView in the IncomingStreamRequestFilter APIs, which will enable us to use IncomingStreamRequestFilter for incoming streams on the exit side too.
* | | | tor-dirserver: Add SaturatingSystemTimeClara Engler2025-10-303-156/+289
| | | | | | | | | | | | | | | | | | | | | | | | This commit changes various things in the dirmirror operation, namely the use of a SystemTime wrapper type that provides saturation on both ends.
* | | | tor-dirserver: Ensure positive timestampsClara Engler2025-10-301-1/+5
| | | | | | | | | | | | | | | | | | | | This commit adds `CHECK` constraints to the database schema in order to ensure all timestamps are positive.
* | | | tor-dircommon: Add TODO to DirTolerance::DefaultClara Engler2025-10-301-0/+2
| | | |
* | | | tor-dirserver: Small database::open() improvementsClara Engler2025-10-301-115/+58
| | | |
* | | | tor-dirserver: Make database full syncClara Engler2025-10-306-292/+231
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit makes the entire database operations synchronous, thereby replacing deadpool with r2d2. The full motivation is outlined in a rustdoc comment at the top of the `database` module, but it can be summarized to the fact that SQLite is by design inherently synchronous due to directly interfacing with the file system.
* | | | tor-dirserver: Outline the dirmirror operationClara Engler2025-10-302-147/+260
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit creates a function named `mirror::operation::serve`, that will serve as the primary entry point for the dirmirror operation. While the function itself already contains some logic, such as querying the timeout to wait for the next consensus, it is largely a rustdoc design document that adds lots of comment to the `mirror::operation` module.
* | | | tor-dirserver: Implement download timeoutClara Engler2025-10-304-0/+450
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit starts work in the `mirror::operation` module by implementing the functions for calculating the `Duration` to wait from a given `SystemTime` until the point in time when the dirmirror should download new documents from the authorities again. Also, this commit introduces new error types and internal helper functions in order to fulfill this purpose better.
* | | | tor-dirserver: Implement read_tx and rw_txClara Engler2025-10-302-55/+352
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit does a large overhaul in the overall way on how we interact with SQLite. Although SQLite is a fairly straightforward tool, it has one large caveat, namely the handling of the infamous `SQLITE_BUSY` error. In WAL mode, the journaling mode we are using, the respective error may occur if there is an existing write transaction while another thread is trying to either upgrade an existing read transaction or create an exclusive transaction at the same time. In this case, SQLite will fail at the statement that either requires the write operation or at the creation of the transaction respectively. In order to tackle this problem, SQLite provides the `busy_timeout` pragma, which allows specifying a timeout in milliseconds during which SQLite will retry to obtain a write transaction or fail immediately. This only works in the latter case, that is, creating a write transaction from the beginning. To solve this issue, we introduce two new functions: 1. `database::read_tx` 2. `database::rw_tx` Both functions accept a database pool alongside a closure accepting a `Transaction` and returning an arbitrary return value. The first function creates a deferred transaction that gets rolled-back in the end, making it suitable for read-only connections. The latter function creates an exclusive transaction that gets committed in the end, making it suitable for read-write connections. This also honors the `busy_timeout`, i.e. the function retries for up to 1s to acquire a write transaction before failing ultimately.
* | | | tor-dirmirror: Introduce the `sql!` macroClara Engler2025-10-303-2/+23
| | | | | | | | | | | | | | | | | | | | | | | | This commit introduces the no-op sql macro used to mark string literals as SQL statements. It is purely semantical and serves the purpose to quickly identify string literals as SQL statements.
* | | | tor-dirserver: Outline the `mirror` APIClara Engler2025-10-302-1/+150
| | | | | | | | | | | | | | | | | | | | | | | | This commit introduces the `mirror` module which is fairly boilerplate at the moment. However, everything of it is very well documented using rustdoc, thereby more or less creating a design document.
* | | | maint: remove all semver.md filesSteven Engler2025-10-305-33/+0
| | | |
* | | | Merge branch 'release-3' into 'main'arti-v1.7.0Nick Mathewson2025-10-291-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | arti-client: Update `ARTI_CLIENT_RELEASE_DATE` See merge request tpo/core/arti!3413
| * | | | arti-client: update `ARTI_CLIENT_RELEASE_DATE`Steven Engler2025-10-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ```bash ./maint/update-release-date ```
* | | | | equix-bench: ran `cargo update`Steven Engler2025-10-291-87/+65
| | | | |
* | | | | hashx-bench: ran `cargo update`Steven Engler2025-10-291-86/+64
| | | | |
* | | | | maint: bump minor versions of all published cratesSteven Engler2025-10-2962-567/+567
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | ```bash readarray -t publish < <(cargo metadata --format-version 1 | jq -r '.packages[] | select((.id | startswith("path+file:///")) and (.rust_version != null) and (.publish == null or .publish == true or .publish != [])) | .name') for package in "${publish[@]}"; do echo "$package:"; cargo set-version --bump minor -p "$package"; done ```
* | | | Merge branch 'remove_socks' into 'main'Nick Mathewson2025-10-286-13/+38
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | arti: Revise strings that implied that we were a SOCKS-only proxy. Closes #2225 See merge request tpo/core/arti!3398
| * | | | arti: explain APP_STREAM_BUF_LEN is what it is.Nick Mathewson2025-10-281-0/+6
| | | | |
| * | | | proto: Adjust a comment about a buffer size.Nick Mathewson2025-10-281-3/+3
| | | | |
| * | | | arti: Revise strings that implied that we were a SOCKS-only proxy.Nick Mathewson2025-10-285-7/+19
| | | | | | | | | | | | | | | | | | | | Closes #2225
| * | | | arti: Stop using SOCKS_BUF_LEN for non-SOCKS proxies.Nick Mathewson2025-10-281-3/+10
| | | | |
* | | | | Merge branch 'futures-copy' into 'main'Nick Mathewson2025-10-288-0/+1406
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | New futures-copy crate to replace copy_interactive. See merge request tpo/core/arti!3393
| * | | | | futures-copy: Add a test for an interactive protocolNick Mathewson2025-10-283-9/+118
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes sure that flushes happen correctly by relaying traffic over a pair of buffered connections between two parties that don't say anything until they receive a message.
| * | | | | New futures-copy crate to replace copy_interactive.Nick Mathewson2025-10-288-0/+1297
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This crate's API is loosely based on some of tokio's functions, and its implementation strategy is loosely based on futures::io::copy. It's designed to be generally useful outside of the arti ecosystem. Should help with the original challenge of #786.
* | | | | arti-ureq: Add MSRV in Cargo.tomlnield2025-10-292-0/+2
| | | | |
* | | | | Merge branch 'release-2' into 'main'Nick Mathewson2025-10-283-0/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | maint: Fixup features for release See merge request tpo/core/arti!3400
| * | | | | maint: fixup featuresSteven Engler2025-10-283-0/+3
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | ```bash cargo run -p fixup-features -- --exclude examples/ --exclude maint/ Cargo.toml ```
* | | | | tor-dirserver: add missing semver breaking changesSteven Engler2025-10-281-0/+3
| | | | |
* | | | | tor-linkspec: add missing semver breaking changesSteven Engler2025-10-281-1/+3
|/ / / /
* / / / arti: Remove now-needless cognitive_complexity exceptionsNick Mathewson2025-10-282-2/+0
|/ / / | | | | | | | | | | | | | | | | | | Apparently the http_connect refactoring made these no longer trigger. Closes #2226
* | | Merge branch 'http_connect_v2' into 'main'Nick Mathewson2025-10-2811-955/+1881
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Implement HTTP CONNECT proxy support in Arti. Closes #2221 See merge request tpo/core/arti!3391
| * | | http_connect: Remove (most) X- prefixes.Nick Mathewson2025-10-282-32/+44
| | | | | | | | | | | | | | | | | | | | | | | | We can't remove them all, since X-Tor-Stream-Isolation is recognized by C-Tor. I've added a non-deprecated Tor-Stream-Isolation that works indepenently.
| * | | http_connect: write our own hyper-futures replacementNick Mathewson2025-10-282-6/+79
| | | |
| * | | http_connect: Forbid non-empty OPTIONS bodies.Nick Mathewson2025-10-281-0/+8
| | | |
| * | | http_connect: validate casei behavior of HeaderMap.Nick Mathewson2025-10-281-0/+36
| | | |
| * | | proxy: Revise rpc docs so they are not SOCKS-specificNick Mathewson2025-10-281-14/+20
| | | |
| * | | http_connect: Advertise listener capabilities to RPCNick Mathewson2025-10-282-7/+25
| | | |
| * | | http_connect: Declare that error messages are text/plain.Nick Mathewson2025-10-281-1/+1
| | | |
| * | | http_connect: Reject OPTIONS requests with bodiesNick Mathewson2025-10-281-2/+11
| | | |
| * | | http_connect: Implement stream isolationNick Mathewson2025-10-283-5/+84
| | | |
| * | | http_connect: implement rpc supportNick Mathewson2025-10-281-6/+76
| | | |
| * | | http-connect: Implement X-Tor-Family-PreferenceNick Mathewson2025-10-281-5/+79
| | | |
| * | | TorAddr: Add function to inspect the host as an IpAddr, if it is one.Nick Mathewson2025-10-281-0/+8
| | | |