mirrors/arti.git/crates/tor-dirmgr/src/retry.rs, branch arti-v1.1.6 mirror of https://gitlab.torproject.org/tpo/core/arti http://git.dilluti0n.com/mirrors/arti.git/atom?h=arti-v1.1.6 2023-01-27T13:28:02Z Allow clippy::unchecked_duration_subtraction in tests 2023-01-27T13:28:02Z Nick Mathewson nickm@torproject.org 2023-01-27T13:01:45Z urn:sha1:a6dd92843edf2bd4c58de62a0d562921e4191ada This panics on error, and we're fine with a panic on misbehavior in tests. test lint blocks: Add many many automatically 2022-12-12T18:00:30Z Ian Jackson ijackson@chiark.greenend.org.uk 2022-12-12T17:58:40Z urn:sha1:125a455bdcca841931f33f83f206e57637ae85c4 This is precisely the result of running the rune in maint/adhoc-add-lint-blocks. impl_standard_builder: Use for tor_dirmgr::DownloadSchedule 2022-06-16T14:42:06Z Ian Jackson ijackson@chiark.greenend.org.uk 2022-06-16T14:17:32Z urn:sha1:17caebd2847e5362106f05d7e758c258ae8b27e0 Add some missing `serde(default)` to uses of humantime_serde::option 2022-06-16T10:08:16Z Ian Jackson ijackson@chiark.greenend.org.uk 2022-06-16T10:07:32Z urn:sha1:4c65e9994d4466f00cd46e0c5a45a1755d3b583c Without this, if the substructure is provided, these fields are mandatory within it. Use testing_rng() in tests throughout our crates. 2022-06-02T18:56:42Z Nick Mathewson nickm@torproject.org 2022-06-02T17:31:43Z urn: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. config derive attrs: Make builders serde, and validated structs not 2022-05-05T09:35:52Z Ian Jackson ijackson@chiark.greenend.org.uk 2022-05-04T16:45:25Z urn:sha1:d47e94b45993162fece0d1bdfa4d3da0407a4861 * Builders additionally derive: Debug, Serialize, Deserialize. * Validated structs no longer derive: Serialize, Deserialize and all related attributes deleted. * As a consequence, all the `#[serde(deny_unknown_fields)]` are gone. That means that right now unknown fields are totally ignored. This is good for compatibility but poor for useability. Doing something better here is arti#417, in progress. * As a consequence, delete tor_dirmgr::retry::default_parallelism. (The default value was already duplicated into a builder attr.) Merge branch 'download-schedule' into 'main' 2022-04-26T18:47:08Z Nick Mathewson nickm@torproject.org 2022-04-26T18:47:08Z urn:sha1:5484bcc21f89d19d30e6351670c36359c132f718 DownloadSchedule: Introduce Builder See merge request tpo/core/arti!473 Make DownloadScheduleBuilder "alternative" defaults pub(crate) 2022-04-26T18:01:20Z Ian Jackson ijackson@chiark.greenend.org.uk 2022-04-26T17:59:10Z urn:sha1:4a32bcd4d3870b2ea5137d6ffeb70bebadcf5ccb As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/473#note_2798464 Fix typo in message 2022-04-26T17:40:05Z Nick Mathewson nickm@torproject.org 2022-04-26T17:40:05Z urn:sha1:4643e1dd249841f590e441cb9a9e8ef6ba750cba DownloadSchedule: Rename field to "attempts" 2022-04-26T14:16:32Z Ian Jackson ijackson@chiark.greenend.org.uk 2022-04-26T14:14:26Z urn:sha1:ed718581233b40a159716f34d56a46c58ebee651 This is actually a number of *attempts* not a number of *retries*. The setter method was already called "attempts". This chnages the deserialisation of the config.