mirrors/arti.git/crates/tor-dirclient, branch tor-dirmgr-v0.5.1

Improve display for tor-dircache errors.

2022-07-07T18:07:04Z

These errors no longer use Debug to dump the `Option`, but instead produce reasonable text. Also, I've fixed the SourceInfo Display implementation so that it now says that it got the error "from $source via $circuit" rather than the other way around.

dirclient: Generalize MdSha256Empty to EmptyRequest.

2022-07-07T18:07:04Z

(It makes sense to use this for things that are not in fact lists of SHA256 digests of Microdescriptors.)

Bump crate and dependency versions.

2022-06-24T16:00:22Z

These were done with the following commands: ``` cargo set-version -p tor-basic-utils --bump patch cargo set-version -p fs-mistrust --bump minor cargo set-version -p tor-error --bump patch cargo set-version -p tor-config --bump patch cargo set-version -p tor-units --bump patch cargo set-version -p tor-rtcompat --bump minor cargo set-version -p tor-llcrypto --bump patch cargo set-version -p tor-bytes --bump minor cargo set-version -p tor-socksproto --bump minor cargo set-version -p tor-cert --bump minor cargo set-version -p tor-cell --bump minor cargo set-version -p tor-proto --bump minor cargo set-version -p tor-netdoc --bump patch cargo set-version -p tor-netdir --bump minor cargo set-version -p tor-persist --bump patch cargo set-version -p tor-chanmgr --bump minor cargo set-version -p tor-guardmgr --bump minor cargo set-version -p tor-circmgr --bump patch cargo set-version -p tor-dirclient --bump patch cargo set-version -p tor-dirmgr --bump minor cargo set-version -p arti-client --bump patch cargo set-version -p arti --bump minor cargo set-version -p arti-bench --bump minor cargo set-version -p arti-testing --bump minor ```

Run maint/add_warning crates/*/src/{lib,main}.rs

2022-06-23T18:15:42Z

Update all lint blocks

Do not include error source() in display() format.

2022-06-21T18:14:14Z

According to doc/Errors.md, and in keeping with current best practices, we should not include display an error's `source()` as part of that error's display method. Instead, we should let the caller decide to call source() and display that error in turn. Part of #323.

dirclient: Fix comments re conssensus request without authorities

2022-06-10T10:25:34Z

dirclient: Replace four very similar "ids in request"

2022-06-09T14:05:32Z

In reviewing !553 I noticed that the empty digest list error had to be handled in two places. I filed #492 about the duplication. In fact it turns out to have been quadruplication. The new code also avoids cloning the underlying objects, instead sorting a Vec of references.

dirclient: Test consensus request based on empty ConsensusRequest

2022-06-09T14:03:44Z

I'm not sure this is right. But providing a test case for it before my next change ensures that my refactoring doesn't change the behaviour.

Merge branch 'lint' into 'main'

2022-05-31T18:27:53Z

lints: Make lint blocks consistent and ensure they stay that way Closes #469 See merge request tpo/core/arti!557

lints: Add let_unit_value allow to all crates

2022-05-31T14:23:52Z

From running add_warning, with manual picking of the right hunks/lines.