summaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | Merge branch 'error-report' into 'main'Nick Mathewson2022-02-227-6/+130
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Improve error messages from arti cli See merge request tpo/core/arti!331
| * | | | tor_error::Report: add testsIan Jackson2022-02-221-0/+56
| | | | |
| * | | | tor_error::Report: Add bound to structIan Jackson2022-02-221-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | This is more typing in the definition but it produces much better error messages if you try to Report(foo).to_string()
| * | | | Move program name printing to report_and_exitIan Jackson2022-02-221-3/+8
| | | | |
| * | | | Placate clippy on nightly, properlyIan Jackson2022-02-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | I c&p this lint suppression but from a cli option printed by clippy, and I didn't notice the need to change the - to _.
| * | | | Placate clippy on nightlyIan Jackson2022-02-181-0/+1
| | | | |
| * | | | Bump anyhow minimal version to 1.0.23Ian Jackson2022-02-184-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the first one where anyhow::Error impl AsRef<dyn StdError> We want this because we want to add error reporting functionality which works with all kinds of errors, which means we need an anyhow::Error which can be vieweed as a StdError. (The alternative would be to deref at the call sites of report_and_exit, making it less ergonomic.) anyhow 1.0.23 is from November 2019.
| * | | | Placate rustfmtIan Jackson2022-02-181-4/+10
| | | | | | | | | | | | | | | | | | | | I think all of this diff is a strict disimprovement, but whatever.
| * | | | Placate clippyIan Jackson2022-02-182-2/+5
| | | | |
| * | | | Add missing docs for report_and_exitIan Jackson2022-02-181-0/+3
| | | | |
| * | | | Add a missing anyhow context() callIan Jackson2022-02-181-2/+2
| | | | |
| * | | | Provide error reporter and use it in the arti binaryIan Jackson2022-02-184-1/+49
| | | | |
* | | | | Merge branch 'pending_error' into 'main'Ian Jackson2022-02-222-47/+65
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | Handle panics from circuit construction Closes #347 See merge request tpo/core/arti!336
| * | | | Handle panics from circuit construction.Nick Mathewson2022-02-182-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We handle them by reporting them to task that's waiting for the circuit, then relaying the panic. Doing so allows the waiting task to distinguish panics (EK::Internal) from cases where the reactor dropped the task entirely (EK::ReactorShuttingDown). And doing _that_ removes one case of EK::Canceled, which helps us on our goals towards #348. Closes #347.
| * | | | Move the main body of our circuit-launching task into a new functionNick Mathewson2022-02-181-47/+55
| | |/ / | |/| | | | | | | | | | This reduces our nesting, and will help us handle panics.
* | | | Merge branch 'proxy-sooner' into 'main'eta2022-02-181-10/+16
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | arti cli: socks proxy: Start listening immedately See merge request tpo/core/arti!333
| * | | | arti cli: Add some anyhow context() callsIan Jackson2022-02-181-5/+8
| | | | |
| * | | | arti cli: socks proxy: Start listening immedatelyIan Jackson2022-02-181-4/+7
| | | | | | | | | | | | | | | | | | | | This makes arti less awkward to use.
| * | | | arti cli: Do config watch setup before entering future selectIan Jackson2022-02-181-3/+3
| |/ / / | | | | | | | | | | | | IMO this clarifies things a bit, and makes things more deterministic.
* / / / Fix typosDimitris Apostolou2022-02-181-2/+2
|/ / /
* | | Merge branch 'script-filenames' into 'main'eta2022-02-181-0/+0
|\ \ \ | |/ / |/| | | | | | | | Remove extension from scripts See merge request tpo/core/arti!328
| * | rename *.py scripts to bare nameIan Jackson2022-02-181-0/+0
| | | | | | | | | | | | | | | | | | Update all references. There are three remaining hits for git-grep '\.py\b' all of which are scripts in "little-t-tor", not here in arti.
* | | Merge branch 'no-http-status-err' into 'main'Ian Jackson2022-02-183-9/+18
|\ \ \ | |/ / |/| | | | | | | | | | | | | | dirclient: Remove HttpStatus error variant Closes #349 See merge request tpo/core/arti!329
| * | dirclient: Remove HttpStatus error variantNick Mathewson2022-02-173-9/+18
| | | | | | | | | | | | | | | | | | | | | Getting a non-200 status is no longer a failure condition; it's just a different kind of answer. Closes #349.
* | | spelling fixNick Mathewson2022-02-171-1/+1
| | |
* | | Add #[display] annotations for remaining ErrorKinds.Nick Mathewson2022-02-171-0/+2
|/ /
* | Merge branch 'remaining-errors'Nick Mathewson2022-02-1718-153/+508
|\ \
| * | Use TorConnectionFailed for failure to download directory.Nick Mathewson2022-02-172-9/+1
| | |
| * | Clarify state vs cache in ErrorKindNick Mathewson2022-02-171-4/+20
| | |
| * | Correct ErrorKinds for some tor-dirclient errors.Nick Mathewson2022-02-172-2/+10
| | |
| * | Rename CircuitTimeout to TorNetworkTimeout.Nick Mathewson2022-02-172-6/+7
| | |
| * | Rename ExitTimeout to RemoteNetworkTimeout.Nick Mathewson2022-02-174-4/+4
| | |
| * | tor_circmgr::Error: Sort variants by interesting-ness.Nick Mathewson2022-02-171-37/+37
| | | | | | | | | | | | We can't use discriminants here now, but maybe we can in the future.
| * | Tweak documentation on ErrorKind::TransientNick Mathewson2022-02-171-5/+4
| | |
| * | tor-dirmgr: Use Bug type for sqlite-detected bugsIan Jackson2022-02-171-1/+9
| | | | | | | | | | | | So we get a stack trace
| * | tor-dirmgr: Make sqlite_error_kind take rusqlite::ErrorIan Jackson2022-02-171-32/+34
| | | | | | | | | | | | | | | We're about to reuse this and we'll want it to take the higher-level type. Also it seems more proper like this.
| * | tor-error: splitting/lumping guidelinesIan Jackson2022-02-171-0/+32
| | |
| * | tor-dirclient: Error::HttpError: add a TODO saying to abolishIan Jackson2022-02-171-0/+4
| | |
| * | ErrorKind::RemoteRefused: Expand on description, narrowing scopeIan Jackson2022-02-171-0/+3
| | |
| * | ErrorKind::Csnceled: Expand on descriptionIan Jackson2022-02-171-0/+3
| | |
| * | ErrorKind::CircuitTimeout: Expand on descriptionIan Jackson2022-02-171-1/+2
| | |
| * | ErrorKind::DirectoryExpired: Add possible deprecation notesIan Jackson2022-02-171-0/+6
| | |
| * | tor-circmgr: errors: Use autoconversion for BugIan Jackson2022-02-171-2/+2
| | |
| * | tor_error: a couple of spelling/formatting issuesNick Mathewson2022-02-161-4/+4
| | |
| * | Finally remove ErrorKind::TODO.Nick Mathewson2022-02-161-8/+0
| | | | | | | | | | | | Closes #322.
| * | arti_client: provide Kinds for all errors.Nick Mathewson2022-02-162-1/+26
| | |
| * | dirmgr: Remember where netdocs came from.Nick Mathewson2022-02-165-16/+61
| | | | | | | | | | | | | | | This isn't complete (see TODO), but it's enough to let us report the right ErrorKind if something fails to parse.
| * | dirmgr: eliminate StringParsingError.Nick Mathewson2022-02-164-29/+23
| | | | | | | | | | | | | | | | | | | | | | | | It had too many possible Kinds depending on what kind of string had failed to parse. I decided to use #[source] here instead of #[from], so that we would have to explicitly convert these errors where they show up.
| * | dirmgr: HasKind for internal and sqlite errorsNick Mathewson2022-02-162-13/+54
| | | | | | | | | | | | | | | At first I had thought that all sqlite errors would be internal, but that's not the case.
| * | dirmgr: implement HasKind for the easier variantsNick Mathewson2022-02-162-10/+61
| | |