| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| |/ / / / / / |
|
| |\ \ \ \ \ \
| |/ / / / /
|/| | | | |
| | | | | |
| | | | | | |
Minor improvements prompted by current nightly clippy
See merge request tpo/core/arti!286
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Prompted by clippy::needless_question_mark. Sometimes Ok(r?) is
needed to do automatic error conversion. I assume the lint checks for
that. Anyway, in these cases it's not needed.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Found via clippy::needless_borrow. In some cases I removed needless
`[..]` too. See also:
needless_borrow suggestion doesn't go far enough
https://github.com/rust-lang/rust-clippy/issues/8389
|
| |/ / / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
clippy::needless_borrow quibbles here, IMO correctly. Its suggestion
didn't go far enough: output is a String and a &String can be passed
to write as-is for identical effect.
|
| |\ \ \ \ \
| |_|/ / /
|/| | / /
| | |/ /
| |/| | |
Preparatory work for auto config reload
See merge request tpo/core/arti!284
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
TorClient doesn't need to be wrapped in an Arc any longer, thanks
to other refactoring.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This is by no means our final API, but should represent an
improvement. Here instead of having to specify a list of files and
their is-this-optional status, along with a list of command-line
options, we have a single structure that encapsulates all of that
information.
Two advantages here:
- Callers no longer have to remember what the boolean means.
- We can "reload" more easily, by keeping the source object around.
This change also implements the correct behavior for our default
configuration file in `arti::main`: if the file is absent and the
user doesn't list a config file, that's no problem. But if the user
lists _that very same config file, we should insist that it be
present.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This enum is required to use `TorClient::reconfigure` correctly, and
as such ought to be re-exported.
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
arti-bench: summarize statistics with a new `Statistic` type (#298)
See merge request tpo/core/arti!279
|
| |/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This implements the proposal from arti#298, making the
`BenchmarkResults` type be made out of a bunch of new `Statistic` types
(which summarize the mean, median, range, and standard deviation of an
arbitrary value) instead of overloading `TimingSummary` for this
purpose.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Fix invalid path character on windows
See merge request tpo/core/arti!277
|
| |/ / / |
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
CI: Add "rustup show" to show compiler versions in logs
See merge request tpo/core/arti!276
|
| | | |
| | |
| | |
| | |
| | |
| | | |
Trying to debug the nightly failure here
https://gitlab.torproject.org/Diziet/arti/-/pipelines/25417
and I can't seem to repro it.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
add postprocessing script to deduplicate cobertura reports
See merge request tpo/core/arti!272
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
tor-netdir needs to bump because tor-netdoc bumped, even though
there were no other changes in tor-netdir. Whoops.
tor-guardmgr needs to bump because it already published, with the
older tor-netdir.
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | |/ /
|/| | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Add shellcheck to gitlab ci and git hook
See merge request tpo/core/arti!275
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| |/ / / |
|
| | | | |
|
| |/ / |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Make max_file_limit configurable
Closes #299
See merge request tpo/core/arti!261
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Improve `coverage` job in CI
See merge request tpo/core/arti!269
|
| |/ / / |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Fix test failure in tor_dirmgr::state::test::get_microdescs_state
See merge request tpo/core/arti!268
|
| |/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This failure occurred because our tests use canned data to exercise
the directory state functionality, and the canned consensus has
suddenly become very expired.
There are better fixes possible, but this is a minimal one that
should get CI working on main again.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Fix/suppress shellcheck errors in teardown.sh
See merge request tpo/core/arti!266
|
| | | | |
| | | |
| | | |
| | | | |
Rationale: It's good to have all of our scripts pass shellcheck.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Unit test cleanup in tor-rtcompat
See merge request tpo/core/arti!267
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Nothing actually used these accessor functions, and it's not clear
what would. We can add them later if they're needed.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
These probably aren't for things that will fail IRL, but it's nice
to have coverage on the code, just in case.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Now there's much less copy-and-paste.
|
| | |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This makes our layout more similar to our other crates, and
successfully informs our grcov exclusion pattern that these tests
are indeed tests.
Doing this knocks down the reported coverage for the tor-rtcompat
crate, but that's okay: we hadn't earned it.
I hereby promise that this commit is only code-movement.
|