summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| | | * | | | | maint/common/forbid-absolute-shebansg: Fix error handlingIan Jackson2024-07-161-4/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | set -o pipefail is defused by the if. And we should properly check the expected statuses from git. If git isn't found, this script would otherwise spuriously think everything is fine.
| | | * | | | | maint/common/apt-install: Don't repeat apt-get updateIan Jackson2024-07-161-1/+9
| | | | | | | |
| | | * | | | | maint/common/apt-install: Use `apt-get install`Ian Jackson2024-07-161-1/+1
| | |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changes from `apt install` (which is not supposed to be used in scripts) to `apt-get install`.
| | | * | | | shellcheck-all: Use -x rather than grepping for included filesIan Jackson2024-10-271-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Empirically, using -x means shellsheck is able to reason more fully about variable settings etc., resulting in fewer false positives. Signed-off-by: Ian Jackson <[email protected]>
| | | * | | | Copy maint/rust-maint-common/forbid-hard-tab from arti.gitIan Jackson2024-10-051-0/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | From commit 62302873e2c0fc640bb75abec73f0b6b4450145d Licence is that of Arti, dual MIT/Apache, so this is fine. Signed-off-by: Ian Jackson <[email protected]>
| | | * | | | shellcheck example: Don't say `./maint`Ian Jackson2024-07-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is unnecessary, and it clutters the .gitlab-ci.yml. Signed-off-by: Ian Jackson <[email protected]>
| | | * | | | for-every-commit: Exclude the subtree(s)Ian Jackson2024-07-191-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise things go quite pear-shaped. Signed-off-by: Ian Jackson <[email protected]>
| | | * | | | for-every-commit: Use array for [t]revlistIan Jackson2024-07-191-4/+4
| | |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is going to contain multiple words, in a moment. Signed-off-by: Ian Jackson <[email protected]>
* | | | | | Merge branch 'bug1885a' into 'main'Nick Mathewson2025-11-043-11/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Stop using MockSleepProvider in a few crates See merge request tpo/core/arti!3424
| * | | | | | Remove one stray allow(deprecated)Neel Chauhan2025-11-041-1/+0
| | | | | | |
| * | | | | | Stop using MockSleepProvider in a few cratesNeel Chauhan2025-11-023-10/+6
| | |_|/ / / | |/| | | | | | | | | | | | | | | | Part of #1885.
* | | | | | Merge branch 'msrv-ci-check' into 'main'Nick Mathewson2025-11-042-0/+35
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | ci-check-msrv: Add check for MSRV See merge request tpo/core/arti!3405
| * | | | | | ci-check-msrv: CI check for MSRV declaration in cratesnield2025-10-292-0/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Checks if each crate has rust-version property in Cargo.toml.
* | | | | | | Merge branch 'refactor-get-recent-consensus' into 'main'Clara Engler2025-11-041-19/+16
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-dirserver: Refactor cons retrieval with JOIN See merge request tpo/core/arti!3436
| * | | | | | | tor-dirserver: Refactor cons retrieval with JOINClara Engler2025-11-041-19/+16
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit refactors `get_recent_consensus()` in `tor-dirserver` to use a single SQL statement with the power of `INNER JOIN` instead of two SQL statements querying the store after retrieving the SHA256 hash. The primary motivation for this is to have cleaner code. A single SQL statement is often-times easier to comprehend, as well more performant in execution, although that can only be proven with a profiler and is not of much concern, given that this function is only executed roughly every 60-90min.
* | | | | | | Merge branch 'bad_extend_test' into 'main'opara2025-11-043-13/+26
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tweak bad_extend_test_impl and its users for reliability and useful errors. See merge request tpo/core/arti!3433
| * | | | | | | bad_extend_test_impl: Wait for message instead of a 100 ms delay.Nick Mathewson2025-11-041-8/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Possible fix for #2232.
| * | | | | | | proto: use assert_matches in circuit.rs tests.Nick Mathewson2025-11-043-5/+8
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 'vim_gitignore' into 'main'Nick Mathewson2025-11-041-0/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Exclude vim *.swp files in .gitignore See merge request tpo/core/arti!3437
| * | | | | | | Exclude vim \*.swp files in .gitignoreNeel Chauhan2025-11-041-0/+1
| |/ / / / / /
* | | | | | | Merge branch 'raise-thresholds' into 'main'Jim Newsome2025-11-041-36/+30
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | shadow CI: increase success threshold for onion svc xfers See merge request tpo/core/arti!3428
| * | | | | | shadow CI: increase success threshold for onion svc xfersJim Newsome2025-11-041-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It looks like the reliability has increased at some point. Let's ratchet up the required-success threshold to avoid silent regression. See <https://gitlab.torproject.org/tpo/core/arti/-/issues/2109#note_3278828>
| * | | | | | shadow CI: add check_host_xfers helper function and improve outputJim Newsome2025-11-041-36/+30
|/ / / / / /
* | | | | | Merge branch 'bump-shadow-2025-11' into 'main'Jim Newsome2025-11-032-3/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | CI: bump shadow commit See merge request tpo/core/arti!3431
| * | | | | | CI: bump shadow commitJim Newsome2025-11-031-2/+2
| | | | | | |
| * | | | | | shadow CI: Use ./ prefix for logging pathJim Newsome2025-11-031-1/+1
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Workaround for arti#2240, though doesn't hurt to be more explicit in general. This bug wasn't triggered in versions of shadow before <https://github.com/shadow/shadow/issues/3659> was fixed - shadow's implementation of `statx` and other syscalls incorrectly behaved as if the `AT_EMPTY_PATH` were set.
* | | | | | Merge branch 'use_futures_copy' into 'main'Nick Mathewson2025-11-037-187/+63
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | Replace copy_interactive with futures-copy. See merge request tpo/core/arti!3416
| * | | | | Replace copy_interactive with futures-copy.Nick Mathewson2025-10-307-187/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | | | | Merge branch 'release-2' into 'main'Nick Mathewson2025-10-311-6/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | ci: Container image downgrade and comment fix for 'cargo-miri' job See merge request tpo/core/arti!3419
| * | | | | | ci: image downgrade and comment fix for 'cargo-miri' jobSteven Engler2025-10-311-6/+6
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | This downgrades the container image for the 'cargo-miri' job, and updates some adjacent comments.
* | | | | | Merge branch 'release-3' into 'main'Nick Mathewson2025-10-316-475/+435
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | doc: Update dependencies See merge request tpo/core/arti!3417
| * | | | | | 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 ```
| * | | | | | maint: downgrade generic-array to 0.14.7Steven Engler2025-10-301-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ```bash cargo update generic-array --precise 0.14.7 ``` See https://gitlab.torproject.org/tpo/core/arti/-/issues/2234 and https://github.com/RustCrypto/traits/issues/2036.
| * | | | | | maint: ran `cargo update`Steven Engler2025-10-301-468/+427
| | | | | | |
| * | | | | | doc: add 'perl-Time-Piece' as a required dependencySteven Engler2025-10-301-1/+1
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For building openssl. This was required by some recent openssl version, but I just noticed it now.
* | | | | | Merge branch 'release-2' into 'main'Nick Mathewson2025-10-313-15/+14
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | ci: Upgrade container versions See merge request tpo/core/arti!3418
| * | | | | maint: update 'extract-md-links' for mistune 3.xSteven Engler2025-10-311-6/+5
| | | | | |
| * | | | | ci: upgrade container versionsSteven Engler2025-10-312-9/+9
|/ / / / /
* | | | | Merge branch 'circ-sync-view-3' into 'main'Nick Mathewson2025-10-308-11/+54
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | proto: Add a new CircSyncView type See merge request tpo/core/arti!3414
| * | | | | 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.
* | | | | Merge branch 'dirmirror-op' into 'main'Clara Engler2025-10-3010-272/+1312
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tor-dirserver: Start working on mirror operation See merge request tpo/core/arti!3301
| * | | | | tor-dirserver: Add SaturatingSystemTimeClara Engler2025-10-304-156/+290
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| | | | | |
| * | | | | Revert "maint/downgrade_dependencies: No longer mess with [email protected]"Clara Engler2025-10-301-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit d37824953f9c530c57c8c5936c12b16a829778c3. Unfortunately, this merge request introduces futures-lite again into the Cargo.lock generated by maint/downgrade_dependencies. Without this, it once again caused the same failures in async-fs and async-global-executor, that were previously fixed before the above commit landed into main. This bug is apparently not reproducible on current git main, because the downgraded Cargo.lock lacks respective dependencies that get transitively introduced by this (arti!3301) merge request.
| * | | | | tor-dirserver: Make database full syncClara Engler2025-10-307-346/+265
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.