mirrors/arti.git/crates/tor-circmgr/src/usage.rs, branch tor-dirmgr-v0.5.1mirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=tor-dirmgr-v0.5.12022-06-13T13:26:32Ztor-netdir: testnet: Make construct_netdir infallible (rustfmt)2022-06-13T13:26:32ZIan Jacksonijackson@chiark.greenend.org.uk2022-06-13T13:26:32Zurn:sha1:3e035927f20e88fa01f3b82649daa08551c4fda6
Run rustfmt. Separate commit to make review of the substantive commit
easier.
tor-netdir: testnet: Make construct_netdir infallible2022-06-13T13:25:45ZIan Jacksonijackson@chiark.greenend.org.uk2022-06-13T13:25:45Zurn:sha1:7d223ac9e6c6d998d51a2d66baaeddea086dca37
This is a *lot* of unwraps. The function takes no parameters and
is used only for testing. It ought to be infallible.
Use testing_rng() in tests throughout our crates.2022-06-02T18:56:42ZNick Mathewsonnickm@torproject.org2022-06-02T17:31:43Zurn:sha1:967ea67b7dfc158f29cf449eb09bac89acdee46b
This only affects uses of thread_rng(), and affects them all more or
less indiscriminately. One test does not work with
ARTI_TEST_PRNG=deterministic; the next commit will fix it.
Resolve the new `derive_partial_eq_without_eq` lint.2022-05-23T16:55:37ZNick Mathewsonnickm@torproject.org2022-05-23T16:32:31Zurn:sha1:546ae3000eb179aedc97de30b0ccdb1d45b93395
It's a little overzealous sometimes, but it's mostly to the good.
squash! Bump every crate's edition to 2021.2022-04-25T17:06:26ZNick Mathewsonnickm@torproject.org2022-04-25T16:40:33Zurn:sha1:2f6bc6bdc431fa36de70a167eea777dcf3f60bf2
Remove all `use` statements for `TryFrom` and `TryInto`. These are
now redundant in Rust 2021.
circmgr: Avoid a race condition in circuit usage restriction2022-04-04T15:14:52ZNick Mathewsonnickm@torproject.org2022-03-31T18:51:02Zurn:sha1:86c59dd1f337919023779ad92bb08ca79e876158
We were treating restrict_mut() failures as internal errors, and
using internal errors to represent them. But in fact, these
failures are entirely possible based on timing. Here's how it
happens:
* Two different circuit requests arrive at the same time, and both
notice a pending circuit that they could use.
* The pending circuit completes; both pending requests are notified.
* The first request calls restrict_mut(), and restricts the request
in such a way that the second couldn't use it.
* The second request calls restrict_mut(), and gets a failure.
Because of this issue, we treat these errors as transient failures
and just wait for another circuit.
Closes #427.
(This is not a breaking API change, since `AbstractSpec` is a
crate-private trait.)
Remove allow(clippy::disallowed_methods) lint.2022-03-30T12:55:58ZNick Mathewsonnickm@torproject.org2022-03-30T12:55:58Zurn:sha1:700e491813f84d76446193aef7121a329b774717Merge branch 'no-system-time' into 'main'2022-03-30T12:44:25Zetator@eta.st2022-03-30T12:44:25Zurn:sha1:fd081742fa13e8464ad46123d83617165587d4f4
Don't use SystemTime::now()
Closes #306
See merge request tpo/core/arti!365remove usage of 'token' where it's no longer a token2022-03-27T11:49:08Ztrinity-1686atrinity@deuxfleurs.fr2022-03-27T11:49:08Zurn:sha1:219ad39ecac25124bf73631d8c166840cc152e55implement IsolationHelper for StreamIsolation2022-03-24T23:03:48Ztrinity-1686atrinity@deuxfleurs.fr2022-03-24T21:56:00Zurn:sha1:85fb91deedb2a7cb879038d4c35d4b0eb119bd52
but don't use it in a dyn Isolation context