aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | Fold UnexplainedTaskSpawnFailure into Internal.Nick Mathewson2022-02-221-9/+1
| | | |
| * | | Rename TorShuttingDown to ArtiShuttingDownNick Mathewson2022-02-222-2/+2
| | | |
| * | | Replace RemoteRefused with CircuitRefused.Nick Mathewson2022-02-222-5/+5
| | | | | | | | | | | | | | | | Also document that it's a lower-level error kind.
| * | | Rename RemoteIdMismatch to RelayIdMismatch.Nick Mathewson2022-02-222-3/+3
| | | |
| * | | Fold EK::Canceled into TransientFailureNick Mathewson2022-02-222-11/+7
| | | | | | | | | | | | | | | | Also add some TODO comments in circmgr for future work.
| * | | Fold AlreadyClosed into BadApiUsageNick Mathewson2022-02-222-6/+1
| | | |
| * | | Add TODOs for errorkinds we mean to fix laterNick Mathewson2022-02-221-2/+13
| | |/ | |/| | | | | | | | | | Per #348, These are the cases that we meant to fix down the road, but not in 0.1.0.
* | | Merge branch 'client_builder' into 'main'Nick Mathewson2022-02-235-51/+117
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Make a TorClientBuilder API. Closes #350 See merge request tpo/core/arti!337
| * | | Documentation suggestion from review.Ian Jackson2022-02-231-1/+1
| | | |
| * | | Make a TorClientBuilder API.Nick Mathewson2022-02-185-51/+117
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a defensive API choice to protect against the possibility that we'll want to add a bunch of other non-config options in the future. Closes #350
* | | | Merge branch 'restore_needless_borrow_check' into 'main'eta2022-02-238-9/+0
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | Remove clippy::needless_borrow exception in CI. Closes #310 See merge request tpo/core/arti!338
| * | | Remove clippy::needless_borrow exception in CI.Nick Mathewson2022-02-208-9/+0
| |/ / | | | | | | | | | | | | This exception is no longer necessary now that the underlying CI bug is fixed.
* | | Merge remote-tracking branch 'origin/mr/335'Nick Mathewson2022-02-221-18/+39
|\ \ \
| * | | 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
| |