summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | Edit tor-error comment to reflect shared ideasNick Mathewson2022-02-181-18/+37
| | |/ | |/|
* | | Merge branch 'error-report' into 'main'Nick Mathewson2022-02-228-6/+131
|\ \ \ | | | | | | | | | | | | | | | | 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-185-1/+50
| | | |
* | | | 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-182-11/+19
|\ \ \ | | | | | | | | | | | | | | | | arti cli: socks proxy: Start listening immedately See merge request tpo/core/arti!333
| * | | tests/chutney/setup: Reduce sleepIan Jackson2022-02-181-1/+3
| | | | | | | | | | | | | | | | Now that arti listens immediately, a very short sleep ought to do.
| * | | 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.
* | | Merge branch 'typos' into 'main'eta2022-02-181-2/+2
|\ \ \ | |/ / |/| | | | | | | | Fix typos See merge request tpo/core/arti!332
| * | Fix typosDimitris Apostolou2022-02-181-2/+2
|/ /
* | Merge branch 'script-filenames' into 'main'eta2022-02-1825-31/+31
|\ \ | |/ |/| | | | | Remove extension from scripts See merge request tpo/core/arti!328
| * maint/add_warning: Remove duplicate argv0 from usage messageIan Jackson2022-02-181-1/+1
| |
| * maint/gen_md_links: Add missing +x bitIan Jackson2022-02-181-0/+0
| | | | | | | | This is a program, with a main function, etc.
| * rename *.py scripts to bare nameIan Jackson2022-02-189-6/+6
| | | | | | | | | | | | 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.
| * rename *.sh scripts to bare nameIan Jackson2022-02-1818-25/+25
| | | | | | | | | | | | | | | | | | As per #309 Update all references. There is one remaining hit for git-grep '\.sh\b' namely "build.sh" in maint/reproducible_build, which I think is the build script for osxcross - ie, an external project.
* | 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-1719-153/+510
|\
| * 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
| |