summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | all: run cargo fmtSteven Engler2025-11-0417-17/+17
| | | | | |
| * | | | | all: replace all uses of `futures::task::SpawnExt` with `tor_rtcompat::SpawnExt`Steven Engler2025-11-0437-46/+46
| | | | | |
| * | | | | maint: add clippy warning for `futures::task::SpawnExt`Steven Engler2025-11-041-0/+1
| | | | | |
| * | | | | tor-rtcompat: add our own `SpawnExt` traitSteven Engler2025-11-042-3/+43
|/ / / / / | | | | | | | | | | | | | | | This new trait is compatible with tokio-console.
* | | | | Merge branch 'maint-common-2' into 'main'Ian Jackson2025-11-043-7/+22
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Import changes from upstream rust-maint-common See merge request tpo/core/arti!3438
| * \ \ \ \ Merge commit 'c8451e50f3aeb268cfa6b81e1e32928250ea3aca' into maint-common-2Ian Jackson2025-11-043-7/+22
| |\ \ \ \ \ | | |_|_|/ / | |/| | | |
| | * | | | Merge branch 'from-arti' into 'main'Ian Jackson2025-11-045-5/+5
| | |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Import changes made in arti.git See merge request tpo/core/rust-maint-common!10
| | | * | | | Switch Cargo.toml files to edition 2024.Nick Mathewson2025-11-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | First, run ``` git grep -l "^edition =" | xargs perl -i -pe 's/^edition *=.*/edition = "2024"/;' ``` Second, manually verify that all Cargo.toml files have changed, and nothing else has changed. Third, run cargo fmt again.
| | | * | | | maint: Avoid x-prefix in comparisons.Gabriela Moldovan2025-11-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This applies the fixes suggested by shellcheck (the x-prefixes were causing the `shellcheck` job to fail; see [SC2268]). [SC2268]: https://www.shellcheck.net/wiki/SC2268
| | | * | | | forbid-hard-tabs: Improve doc commentIan Jackson2025-11-041-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2488#note_3087164
| | | * | | | bash-utils.sh: reject_options: Don't crash due to -u if no argumentsIan Jackson2025-11-041-1/+1
| | | | | | |
| | | * | | | Provide new forbid-hard-tabs scriptIan Jackson2025-11-041-0/+45
| | | | | | |
| | | * | | | Fix typosDimitris Apostolou2025-11-041-1/+1
| | | | | | |
| | | * | | | 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