| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| | * | | | Replace TorNetworkError with TorDirectoryError | Nick Mathewson | 2022-02-22 | 2 | -7/+7 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | This is still not as specific as we want; but there's already a TODO comment in tor-dirclient::err about fixing that at some point in the future. | |||||
| | * | | | Rename TorConnectionFailed to TorAccessFailed | Nick Mathewson | 2022-02-22 | 3 | -4/+4 | |
| | | | | | ||||||
| | * | | | Fold UnexplainedTaskSpawnFailure into Internal. | Nick Mathewson | 2022-02-22 | 1 | -9/+1 | |
| | | | | | ||||||
| | * | | | Rename TorShuttingDown to ArtiShuttingDown | Nick Mathewson | 2022-02-22 | 2 | -2/+2 | |
| | | | | | ||||||
| | * | | | Replace RemoteRefused with CircuitRefused. | Nick Mathewson | 2022-02-22 | 2 | -5/+5 | |
| | | | | | | | | | | | | | | | | | Also document that it's a lower-level error kind. | |||||
| | * | | | Rename RemoteIdMismatch to RelayIdMismatch. | Nick Mathewson | 2022-02-22 | 2 | -3/+3 | |
| | | | | | ||||||
| | * | | | Fold EK::Canceled into TransientFailure | Nick Mathewson | 2022-02-22 | 2 | -11/+7 | |
| | | | | | | | | | | | | | | | | | Also add some TODO comments in circmgr for future work. | |||||
| | * | | | Fold AlreadyClosed into BadApiUsage | Nick Mathewson | 2022-02-22 | 2 | -6/+1 | |
| | | | | | ||||||
| | * | | | Add TODOs for errorkinds we mean to fix later | Nick Mathewson | 2022-02-22 | 1 | -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 Mathewson | 2022-02-23 | 5 | -51/+117 | |
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | Make a TorClientBuilder API. Closes #350 See merge request tpo/core/arti!337 | |||||
| | * | | | Documentation suggestion from review. | Ian Jackson | 2022-02-23 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | Make a TorClientBuilder API. | Nick Mathewson | 2022-02-18 | 5 | -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' | eta | 2022-02-23 | 8 | -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 Mathewson | 2022-02-20 | 8 | -9/+0 | |
| | |/ / | | | | | | | | | | | | | This exception is no longer necessary now that the underlying CI bug is fixed. | |||||
| * | | | Merge remote-tracking branch 'origin/mr/335' | Nick Mathewson | 2022-02-22 | 1 | -18/+39 | |
| |\ \ \ | ||||||
| | * | | | Edit tor-error comment to reflect shared ideas | Nick Mathewson | 2022-02-18 | 1 | -18/+37 | |
| | | |/ | |/| | ||||||
| * | | | Merge branch 'error-report' into 'main' | Nick Mathewson | 2022-02-22 | 7 | -6/+130 | |
| |\ \ \ | | | | | | | | | | | | | | | | | Improve error messages from arti cli See merge request tpo/core/arti!331 | |||||
| | * | | | tor_error::Report: add tests | Ian Jackson | 2022-02-22 | 1 | -0/+56 | |
| | | | | | ||||||
| | * | | | tor_error::Report: Add bound to struct | Ian Jackson | 2022-02-22 | 1 | -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_exit | Ian Jackson | 2022-02-22 | 1 | -3/+8 | |
| | | | | | ||||||
| | * | | | Placate clippy on nightly, properly | Ian Jackson | 2022-02-18 | 1 | -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 nightly | Ian Jackson | 2022-02-18 | 1 | -0/+1 | |
| | | | | | ||||||
| | * | | | Bump anyhow minimal version to 1.0.23 | Ian Jackson | 2022-02-18 | 4 | -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 rustfmt | Ian Jackson | 2022-02-18 | 1 | -4/+10 | |
| | | | | | | | | | | | | | | | | | I think all of this diff is a strict disimprovement, but whatever. | |||||
| | * | | | Placate clippy | Ian Jackson | 2022-02-18 | 2 | -2/+5 | |
| | | | | | ||||||
| | * | | | Add missing docs for report_and_exit | Ian Jackson | 2022-02-18 | 1 | -0/+3 | |
| | | | | | ||||||
| | * | | | Add a missing anyhow context() call | Ian Jackson | 2022-02-18 | 1 | -2/+2 | |
| | | | | | ||||||
| | * | | | Provide error reporter and use it in the arti binary | Ian Jackson | 2022-02-18 | 4 | -1/+49 | |
| | | | | | ||||||
| * | | | | Merge branch 'pending_error' into 'main' | Ian Jackson | 2022-02-22 | 2 | -47/+65 | |
| |\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | | Handle panics from circuit construction Closes #347 See merge request tpo/core/arti!336 | |||||
| | * | | | Handle panics from circuit construction. | Nick Mathewson | 2022-02-18 | 2 | -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 function | Nick Mathewson | 2022-02-18 | 1 | -47/+55 | |
| | | |/ | |/| | | | | | | | This reduces our nesting, and will help us handle panics. | |||||
| * | | | Merge branch 'proxy-sooner' into 'main' | eta | 2022-02-18 | 1 | -10/+16 | |
| |\ \ \ | | | | | | | | | | | | | | | | | arti cli: socks proxy: Start listening immedately See merge request tpo/core/arti!333 | |||||
| | * | | | arti cli: Add some anyhow context() calls | Ian Jackson | 2022-02-18 | 1 | -5/+8 | |
| | | | | | ||||||
| | * | | | arti cli: socks proxy: Start listening immedately | Ian Jackson | 2022-02-18 | 1 | -4/+7 | |
| | | | | | | | | | | | | | | | | | This makes arti less awkward to use. | |||||
| | * | | | arti cli: Do config watch setup before entering future select | Ian Jackson | 2022-02-18 | 1 | -3/+3 | |
| | |/ / | | | | | | | | | | IMO this clarifies things a bit, and makes things more deterministic. | |||||
| * / / | Fix typos | Dimitris Apostolou | 2022-02-18 | 1 | -2/+2 | |
| |/ / | ||||||
| * | | Merge branch 'script-filenames' into 'main' | eta | 2022-02-18 | 1 | -0/+0 | |
| |\ \ | |/ |/| | | | | | Remove extension from scripts See merge request tpo/core/arti!328 | |||||
| | * | rename *.py scripts to bare name | Ian Jackson | 2022-02-18 | 1 | -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 Jackson | 2022-02-18 | 3 | -9/+18 | |
| |\ \ | |/ |/| | | | | | | | | | dirclient: Remove HttpStatus error variant Closes #349 See merge request tpo/core/arti!329 | |||||
| | * | dirclient: Remove HttpStatus error variant | Nick Mathewson | 2022-02-17 | 3 | -9/+18 | |
| | | | | | | | | | | | | | | | Getting a non-200 status is no longer a failure condition; it's just a different kind of answer. Closes #349. | |||||
| * | | spelling fix | Nick Mathewson | 2022-02-17 | 1 | -1/+1 | |
| | | | ||||||
| * | | Add #[display] annotations for remaining ErrorKinds. | Nick Mathewson | 2022-02-17 | 1 | -0/+2 | |
| |/ | ||||||
| * | Merge branch 'remaining-errors' | Nick Mathewson | 2022-02-17 | 18 | -153/+508 | |
| |\ | ||||||
| | * | Use TorConnectionFailed for failure to download directory. | Nick Mathewson | 2022-02-17 | 2 | -9/+1 | |
| | | | ||||||
| | * | Clarify state vs cache in ErrorKind | Nick Mathewson | 2022-02-17 | 1 | -4/+20 | |
| | | | ||||||
| | * | Correct ErrorKinds for some tor-dirclient errors. | Nick Mathewson | 2022-02-17 | 2 | -2/+10 | |
| | | | ||||||
| | * | Rename CircuitTimeout to TorNetworkTimeout. | Nick Mathewson | 2022-02-17 | 2 | -6/+7 | |
| | | | ||||||
| | * | Rename ExitTimeout to RemoteNetworkTimeout. | Nick Mathewson | 2022-02-17 | 4 | -4/+4 | |
| | | | ||||||
| | * | tor_circmgr::Error: Sort variants by interesting-ness. | Nick Mathewson | 2022-02-17 | 1 | -37/+37 | |
| | | | | | | | | | We can't use discriminants here now, but maybe we can in the future. | |||||
| | * | Tweak documentation on ErrorKind::Transient | Nick Mathewson | 2022-02-17 | 1 | -5/+4 | |
| | | | ||||||
