| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| |\
| |
| |
| |
| | |
Disable macos build tests
See merge request tpo/core/arti!2226
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Rename OptTimestamp to AtomicOptTimestamp
Closes #1412
See merge request tpo/core/arti!2218
|
| | |/ |
|
| |\ \
| |/
|/|
| |
| | |
Turn crate ownership discrepancies into a warning
See merge request tpo/core/arti!2220
|
| |/
|
|
|
| |
Ideally we'd tolerate situations with unaccepted ownership requests,
but that information doesn't seem to be public.
|
| |\
| |
| |
| |
| | |
Resolve a rustdoc warning.
See merge request tpo/core/arti!2215
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | | |
Write a README for tor-rpcbase
See merge request tpo/core/arti!2210
|
| | | | |
|
| |\ \ \
| |_|/
|/| |
| | |
| | | |
Better crate ownership check, and run it in CI
See merge request tpo/core/arti!2201
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This is superseded now.
(I wish I had noticed it; perhaps I could have souped it up rather
than writing a whole new thing, but I think the result would have been
much the same.)
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
I think that the effect of this will be as follows:
* MR branches will never run this, so when the crate ownership
is wrong, development can continue.
* When a new crate is first published, it will transition from
"ignored" to "complaining". This will happen on the first
CI run on main after "cargo publish" is run. But it *won't*
happen on the CI run on the publication tag.
The intent is that we discover a failure to `git add` while the
release technician who published the new crate is still around.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Compared to check-crate-owners:
1. Checks for uniformity, rather than having an embedded list
of intended maintainers.
2. Uses curl | jq, rather than `cargo owner`.
This is necessary for:
3. Ignores crates which have never been published.
This is necessary to be able to run this in CI, since there
may be new crates in the workspace.
4. Prints the ownership information rather differently -
just the usernames. This is a consequence of using curl | jq.
|
| | | |
| | |
| | |
| | | |
This is useful for debugging.
|
| | | | |
|
| | | | |
|
| | |/
| |
| |
| |
| | |
We (will) use `*.sh` for shell include files. We're about to make one
of these.
|
| |\ \
| |/
|/|
| |
| |
| |
| | |
TorClientBuilder: Add the ability to retry when fslock fails
Closes #1464
See merge request tpo/core/arti!2198
|
| | | |
|
| | | |
|
| | |
| |
| |
| | |
Name chosen to match the error kind that we're detecting.
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This commit adds a parameter to TorClientBuilder that control how long
we should retry constructing a TorClient if we get a
LocalResourceInUse error. When this parameter is not set, we
default to 500 milliseconds for async entry points and 0
milliseconds for sync entry points.
(`LocalResourceInUse` usually means that a lockfile is held by
somebody else; but when the resource is some other type, we
typically want the same behavior anyway.)
(I really don't want to introduce delays by default for the
create_unbootstrapped case, since it previously had no delay at
all.)
There is now also an async entry point to create an unbootstrapped
TorClient.
Closes #1464.
|
| |/
|
|
|
| |
There is no actual reason to consume this type, and taking it by
reference allows us to retry.
|
| |\
| |
| |
| |
| |
| |
| | |
tor-circmgr: Refactor VanguardHsPathBuilder::pick_path
Closes #1459
See merge request tpo/core/arti!2205
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| |
| |
| | |
This is just code motion: moving the vanguard-specific parts of
`maybe_extend_stub_circuit()` behind the `vanguards` feature will enable
us to refactor it to use `select_middle_for_vanguard_circuit()`, which
is only available if the `vanguards` feature is enabled.
|
| | | |
|
| | |
| |
| |
| |
| |
| | |
This is a follow up from https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2186#note_3035525
Closes #1459
|
| |/
|
|
| |
There's not much to refactor about this line.
|
| |\
| |
| |
| |
| | |
tor-circmgr: Write tests for the HsPathBuilder.
See merge request tpo/core/arti!2199
|
| | | |
|
| | |
| |
| |
| |
| |
| | |
This test is not new (it was added in !2168), but I think it's a good
idea to annotate the tests preventing security issues with the TROVE
number and/or arti ticket they pertain to.
|
| | |
| |
| |
| |
| |
| |
| | |
These tests should give us *some* assurance that the upcoming
`HsVanguardPathBuilder` refactoring doesn't break anything.
Part of #1459
|
| | |
| |
| |
| | |
Part of #1459
|
| | | |
|
| | |
| |
| |
| |
| |
| | |
We will soon need these helpers outside of `tor-guardmgr` too.
This commit is mainly code motion.
|
| |/ |
|
| |\
| |
| |
| |
| |
| |
| | |
Update curve25519-dalek to 4.1.3.
Closes #1468
See merge request tpo/core/arti!2211
|
| |/
|
|
|
|
|
|
|
| |
This updates our `curve25519-dalek` dependency to 4.1.3, which doesn't
have the issues described in RUSTSEC-2024-0344.
Closes TROVE-2024-007 and #1468
Fixes the failing cargo-audit job.
|
| |\
| |
| |
| |
| |
| |
| | |
tls: Support export keying material (RFC 5705)
Closes #1432
See merge request tpo/core/arti!2185
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Add a function to get the keying material as detailed by RFC 5705.
Because native-tls doesn't have such support, there is a place holder
panic!() for now.
This means that for the forseable future, relay would only work with
rustls until we figure out a solution for native-tls.
Closes #1432
Signed-off-by: David Goulet <[email protected]>
|
| |\ \
| | |
| | |
| | |
| | | |
CI: Bail if unexpectedly using a non-amd64 container
See merge request tpo/core/arti!2207
|