summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| | | * | | | Exclude debian/* from the forbid-absolute-shebangs checksNoisyCoil2025-11-041-1/+1
| | | | | | |
| | * | | | | Merge branch 'from-t2usm' into 'main'Ian Jackson2025-11-044-7/+71
| | |\ \ \ \ \ | | | |/ / / / | | |/| | | | | | | | | | | | | | | | | | Import changes made in Debian tag2upload-service-manager See merge request tpo/core/rust-maint-common!9
| | | * | | | shellcheck: Add comment about why we're allowing x-hackIan Jackson2025-11-041-0/+5
| | | | | | |
| | | * | | | shellcheck: disable two unhelpful shellcheck complaintsIan Jackson2025-11-041-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This affects rust-maint-common only.
| | | * | | | Merge new rust-maint-commonIan Jackson2025-11-043-7/+26
| | | |\ \ \ \ | | | |/ / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | Merge commit 'ccb054bfec079f73d54ee74105fb60a56dcb9918' Signed-off-by: Ian Jackson <[email protected]>
| | * | | | | Merge branch 'from-arti' into 'main'Ian Jackson2024-07-223-7/+26
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Merge changes upstream from arti.git See merge request tpo/core/rust-maint-common!8
| | | * | | | | maint/rust-maint-common/forbid-absolute-shebang: Fix error messageIan Jackson2024-07-171-1/+1
| | | | | | | |
| | | * | | | | maint/common/update-shell-includes: Tolerate includes with a `$`Ian Jackson2024-07-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are not hardcoded, whatever they are, so they're not the principal thing we're trying to prevent. Previously, this would trip on this in arti.git:maint/cargo-publish: maint="$(dirname "$0")" ... # shellcheck source=maint/crates-io-utils.sh source "$maint/crates-io-utils.sh" Ideally we'd teach this script to be able to check and maintain includes for scripts that aren't in common/, but I think that's a task for another day.
| | | * | | | | 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.