| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| | * | | | | 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 | |
| | | | | ||||||
| | * | | tor-dirmgr: Use Bug type for sqlite-detected bugs | Ian Jackson | 2022-02-17 | 1 | -1/+9 | |
| | | | | | | | | | | | | | So we get a stack trace | |||||
| | * | | tor-dirmgr: Make sqlite_error_kind take rusqlite::Error | Ian Jackson | 2022-02-17 | 1 | -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 guidelines | Ian Jackson | 2022-02-17 | 1 | -0/+32 | |
| | | | | ||||||
| | * | | tor-dirclient: Error::HttpError: add a TODO saying to abolish | Ian Jackson | 2022-02-17 | 1 | -0/+4 | |
| | | | | ||||||
| | * | | ErrorKind::RemoteRefused: Expand on description, narrowing scope | Ian Jackson | 2022-02-17 | 1 | -0/+3 | |
| | | | | ||||||
| | * | | ErrorKind::Csnceled: Expand on description | Ian Jackson | 2022-02-17 | 1 | -0/+3 | |
| | | | | ||||||
| | * | | ErrorKind::CircuitTimeout: Expand on description | Ian Jackson | 2022-02-17 | 1 | -1/+2 | |
| | | | | ||||||
| | * | | ErrorKind::DirectoryExpired: Add possible deprecation notes | Ian Jackson | 2022-02-17 | 1 | -0/+6 | |
| | | | | ||||||
| | * | | tor-circmgr: errors: Use autoconversion for Bug | Ian Jackson | 2022-02-17 | 1 | -2/+2 | |
| | | | | ||||||
| | * | | tor_error: a couple of spelling/formatting issues | Nick Mathewson | 2022-02-16 | 1 | -4/+4 | |
| | | | | ||||||
