summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Add a few coverage-based tests to tor-config.Nick Mathewson2022-05-314-1/+122
| | | | | | | | | | | | | | | | | | | | There's nothing major here, but it does fill in a few gaps.
* | | | | Merge branch 'dbg' into 'main'Nick Mathewson2022-05-311-0/+5
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | Revert "Remove dbg!()s in tor-config" See merge request tpo/core/arti!552
| * | | | Revert "Remove dbg!()s in tor-config"Ian Jackson2022-05-271-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was done because Nightly Rust complained about these, despite them all being in tests. That is now fixed upstream: https://github.com/rust-lang/rust-clippy/issues/8758 https://github.com/rust-lang/rust-clippy/pull/8838 This reverts commit 9d26a91886990b08dc5b6033c290d417489c61fc.
* | | | | Merge branch 'post-release-fix-20220527' into 'main'Ian Jackson2022-05-302-1/+2
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | Fix minor issues found while publishing crates for Arti 0.4.0 See merge request tpo/core/arti!547
| * | | | tor-rtcompat: Fix a warning when building with no backends.Nick Mathewson2022-05-271-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cargo publish (and probably nobody else!) builds this crate with no features enabled. When you do that, you get a warning about an unused `use std::io`. Fixing that.
| * | | | Move tor-error crate to the "rust-patterns" category.Nick Mathewson2022-05-271-1/+1
| | | | | | | | | | | | | | | | | | | | There is no "tor" category; crates.io warned about this.
* | | | | Merge branch 'bug487' into 'main'Ian Jackson2022-05-301-0/+6
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fs-mistrust: add getegid() to getgrouplist() output. Closes #487 See merge request tpo/core/arti!548
| * | | | | fs-mistrust: add getegid() to getgrouplist() output.Nick Mathewson2022-05-271-0/+6
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a unit test failure in weird environments (like some containers) where the current effective GID is not included in the list of current groups. Closes #487. Bug reported by @sjm217.
* | | | | Merge branch 'arti-cfg-tests' into 'main'Ian Jackson2022-05-301-0/+24
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | add unit tests for ArtiConfig public functions See merge request tpo/core/arti!551
| * | | | | add unit tests for ArtiConfig public functionsmichael mccune2022-05-281-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this change adds some simple tests for the ArtiConfig public getter functions to help expand coverage in this crate.
* | | | | | Merge branch 'chore/integrate_cargo_sort' into 'main'Ian Jackson2022-05-3035-258/+262
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Lexically sort Cargo.toml dependencies Closes #490 See merge request tpo/core/arti!549
| * | | | | | Lexically sort Cargo.toml dependenciesOrhun Parmaksız2022-05-2835-258/+262
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Utilize cargo-sort: https://github.com/DevinR528/cargo-sort Signed-off-by: Orhun Parmaksız <[email protected]>
* | | | | | Merge branch 'per_crate_semver' into 'main'Ian Jackson2022-05-301-13/+35
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update semver_status.md with new procedure. See merge request tpo/core/arti!545
| * | | | | | Update semver_status.md with new procedure.Nick Mathewson2022-05-271-13/+35
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now (per discussion on #471) we use one file per crate. I've tried to be more explicit about the format to use, and what to add when.
* | | | | | Merge branch 'mistrust_osstring_limitation' into 'main'Ian Jackson2022-05-302-4/+13
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | fs-mistrust: Document problems with non-UTF8 OsString in toml See merge request tpo/core/arti!538
| * | | | | | fs-mistrust: Document problems with non-UTF8 OsString in tomlNick Mathewson2022-05-262-4/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It turns out that the `toml` crate can't handle OsString, since `toml` doesn't support serialize_newtype_variant, and the `serde` crate tries to serialize OsString using that method. In this commit we document that limitation, and test that we can at least round-trip through json. Found by inspecting test coverage.
* | | | | | | Merge branch 'dns-dedup' into 'main'Ian Jackson2022-05-301-49/+139
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | deduplicate dns requests based on transaction id Closes #411 See merge request tpo/core/arti!535
| * | | | | | deduplicate dns requests based on transaction idtrinity-1686a2022-05-301-49/+139
|/ / / / / /
* | | | | | Merge branch 'macro' into 'main'Nick Mathewson2022-05-271-0/+3
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | Suppress new "unused_macro_rules" lint in one place See merge request tpo/core/arti!544
| * | | | | tor-netdoc: Allow suppressing unknown lintsIan Jackson2022-05-271-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We want to supress the unused_macro_rules lint in one place but it's new in Nightly and this triggers yet another lint.
| * | | | | tor-netdoc: Suppress new "unused_macro_rules" lint in one placeIan Jackson2022-05-271-0/+1
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | The compiler can apparently tell that we aren't generating compile errors! Also that we're not yet using a feature provided here. (New lint in currently Nightly.)
* | | | | Merge branch 'fix-coverage-help' into 'main'Nick Mathewson2022-05-271-1/+1
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | Fix "coverage -h" command See merge request tpo/core/arti!539
| * | | | Fix "coverage -h" commandSteven Murdoch2022-05-261-1/+1
| | |/ / | |/| | | | | | | | | | $SCRIPT_NAME is not defined so causes an error. Use $(basename "$0") instead.
* | | | Merge branch 'bench-default-chutney' into 'main'Nick Mathewson2022-05-271-3/+11
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | add default chutney path to arti-bench script See merge request tpo/core/arti!541
| * | | | add default chutney path to arti-bench scripttrinity-1686a2022-05-261-3/+11
| | | | |
* | | | | Clear out 0.4.0 semver-statusNick Mathewson2022-05-271-52/+3
| | | | |
* | | | | Regenerate version bump from previous commit.arti-v0.4.0Nick Mathewson2022-05-2718-75/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit was made by reverting the previous commit, then re-running the script I used to generate it. In theory there should be no semantic changes: only changes due to improved formatting from cargo edit.
* | | | | Semantic version changes for Arti 0.4.0 releaseNick Mathewson2022-05-2725-138/+138
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I followed the following procedure to make these changes: * I used maint/changed_crates to find out which crates had changed since 0.3.0. * I used grep and maint/list_crates to sort those crates in topological (dependency) order. * I looked through semver_status to find which crates were listed as having semver-relevant changes (new APIs and breaking changes). * I scanned through the git logs of the crates with no semver-relevant changes listed to confirm that, indeed, they had no changes. For those crates, I incremented their patch-level version _without_ changing the version that other crates depend on. * I scanned through the git logs of the crates with no semver-relevant changes listed to confirm that, indeed, they had no obvious breaking changes. * I treated all crates that depend on `arti` and/or `arti-client` as having breaking changes. * I identified crates that depend on crates that have changed, even if they have not changed themselves, and identified them as having a non-breaking change. * For all of the crates, I used `cargo set-version -p $CRATE --bump $STATUS` (where `STATUS` is `patch` or `minor`) to update the versions, and the depended-upon versions.
* | | | | Bump versions of trivially-changed crates.Nick Mathewson2022-05-278-14/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These crates had only clippy fixes that do not affect their behavior: tor-bytes tor-cell tor-events tor-linkspec tor-netdir tor-socksproto This crate only had the cargo-husky dependency removed, which does not affect compatibility: tor-llcrypto Since these changes have no compatibility effects, it is not necessary to bump the versions of these crates which other crates depend on.
* | | | | Update tor-config README.md.Nick Mathewson2022-05-271-1/+41
| | | | |
* | | | | Run "cargo update" in preparation for releaseNick Mathewson2022-05-271-4/+4
| | | | |
* | | | | Light edits to changelog.Nick Mathewson2022-05-271-2/+2
| | | | |
* | | | | Fix a link with nightly rustdocNick Mathewson2022-05-261-1/+1
|/ / / /
* | | | CHANGELOG: add a blurb.Nick Mathewson2022-05-261-2/+7
| | | |
* | | | CHANGELOG: Acknowledgments and linksNick Mathewson2022-05-261-9/+55
| | | |
* | | | Add new entries to changelog.Nick Mathewson2022-05-261-2/+15
|/ / /
* | | Merge branch 'tests-20220525' into 'main'Nick Mathewson2022-05-263-4/+150
|\ \ \ | |/ / |/| | | | | | | | Add a few coverage-driven tests See merge request tpo/core/arti!536
| * | Remove some stray dbg!() calls.Nick Mathewson2022-05-261-2/+1
| | |
| * | Test a few more corner cases in tor-persistNick Mathewson2022-05-252-4/+111
| | | | | | | | | | | | | | | Also, if we get an IoError from a call to fs-mistrust, report it as an IoError.
| * | More tests for ClockSkew.Nick Mathewson2022-05-251-0/+40
| | |
* | | Merge branch 'link-to-coverage' into 'main'Nick Mathewson2022-05-261-1/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | add link to coverage reports Closes #251 See merge request tpo/core/arti!537
| * | | add link to coverage reportstrinity-1686a2022-05-261-1/+2
| | | |
* | | | Merge branch 'standardise-shell' into 'main'Nick Mathewson2022-05-2621-61/+76
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Standardise shell Closes #425 See merge request tpo/core/arti!533
| * | | | fix docker_reproducible_build not passing its parameters properlytrinity-1686a2022-05-261-1/+1
| | | | |
| * | | | update comments in coverage_fuzz_corporatrinity-1686a2022-05-251-25/+15
| | | | |
| * | | | update readmestrinity-1686a2022-05-253-8/+20
| | | | |
| * | | | make almost all scripts use bash and set euo pipefailtrinity-1686a2022-05-2518-29/+42
| |/ / /
* | | | Merge branch 'config-split' into 'main'Ian Jackson2022-05-2611-85/+862
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Break TorClientConfig out of ArtiConfig and warn on unknown config keys Closes #459 and #417 See merge request tpo/core/arti!529
| * | | | tor-config: load: clippy: adopt two suggestionsIan Jackson2022-05-261-3/+2
| | | | | | | | | | | | | | | | | | | | We can have mem::take, hooray.
| * | | | tor-config: load: Add missing "not".Ian Jackson2022-05-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807331