mirrors/arti.git/crates/tor-dirmgr/src, branch arti-v1.2.7mirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=arti-v1.2.72024-09-03T08:09:02ZFix typos2024-09-03T08:09:02ZDimitris Apostoloudimitris.apostolou@icloud.com2024-09-03T08:09:02Zurn:sha1:676f913463b7d84f34dd6101b99d03ef79cf57adextract tor_async_utils::oneshot into ::oneshot-fused-workaround2024-08-28T14:27:46ZJim Newsomejnewsome@torproject.org2024-08-22T19:20:36Zurn:sha1:46f7f01092e6ac55e3e958dc3a2228b1d36e26a1
Having this in the `tor-async-utils` crate prevents us from doing both
of the following without introducing a circular dependency:
* using it in `tor-rtmock` (which we currently do, particularly in
tests).
* using `tor-rtmock` to test things in `tor-async-utils`. We don't do
this yet, but it is generally sensible to do so. In particular we
want to move the `stream_peak` module there, which is currently tested
with `tor-rtmock`.
Moving this into its own crate avoids this circular dependency.
Merge branch 'sqlite-race' into 'main'2024-07-31T08:59:43Zgabi-250gabi@torproject.org2024-07-31T08:59:43Zurn:sha1:041ef9b3068d281bbcdd32f65b57215f835dbb42
tor-dirmgr: Return an error if storage is readonly and DB is missing/incompatbile.
Closes #1497
See merge request tpo/core/arti!2283tor-dirmgr: Use Path::try_exists() instead of Path::exists().2024-07-30T14:20:38ZGabriela Moldovangabi@torproject.org2024-07-30T14:17:24Zurn:sha1:e2b7ac36df973ef97b3740e51acd9cd1da80a357tor-dirmgr: Replace from_conn impl with a call to from_conn_internal helper.2024-07-30T10:00:48ZGabriela Moldovangabi@torproject.org2024-07-29T15:42:23Zurn:sha1:89331d0862f347427507360c2f7e1a23b1dd8f32tor-dirmgr: Return an error if storage is readonly and DB is missing/incompatbile.2024-07-30T10:00:48ZGabriela Moldovangabi@torproject.org2024-07-23T15:16:21Zurn:sha1:d7b92bd91d117deb6be85f82bfadd1bbae463a2c
This fixes a bug in `SqliteStore`'s constructor: previously, it would
unconditionally try to create the missing database, even if it didn't
have write access. As a result, it was impossible to reliably start
multiple concurrent arti processes configured with the same (empty or
nonexistent) cache_dir, because many of them would fail with errors such
as
```
attempt to write a readonly database: Error code 8: Attempt to write a readonly database
```
Returning a `LocalResourceAlreadyInUse` error kind here enables us to
leverage the retry loop from `TorClientBuilder::create_unbootstrapped`
(which retries on local resource errors if `local_resource_timeout` is
set).
Closes #1497
tor-dirmgr: Deprecate Error::CachePermissions, use CacheAccess2024-07-10T10:50:44ZIan Jacksonijackson@chiark.greenend.org.uk2024-07-09T13:35:50Zurn:sha1:19fd35e1a3936929a88f077c58019189323044b0tor-dirmgr: Correct message and description for mistrust error2024-07-10T10:50:44ZIan Jacksonijackson@chiark.greenend.org.uk2024-07-09T13:32:20Zurn:sha1:1a29495316b94bba214dc3970d23d5d6e5110f7btor-dirmgr: Add some duplicate dead code allows2024-07-08T10:36:25ZIan Jacksonijackson@chiark.greenend.org.uk2024-07-08T10:22:49Zurn:sha1:168cfc2c6e42182a703e51ea61b50dc82a2dfbe4
Sadly, rustc seems to want us to mark this allow in several places.
Merge branch 'tolerate_missing_blob' into 'main'2024-06-18T14:26:40ZNick Mathewsonnickm@torproject.org2024-06-18T14:26:40Zurn:sha1:291b8bfc4dfad6a7de8e76d8437c90703c10e750
dirmgr::storage: Treat a missing blob file as an absent object.
Closes #1466
See merge request tpo/core/arti!2200