aboutsummaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | tor-proto: Split IoErr based on when it occursNick Mathewson2022-02-235-21/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We want to distinguish handshake failures from errors later on in the channel's lifetime. Closes #359.
* | | | Merge branch 'remote_stream_err' into 'main'eta2022-02-243-4/+75
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Give specific error kinds to different END reasons Closes #360 See merge request tpo/core/arti!343
| * | | | Give specific error kinds to different END reasonsNick Mathewson2022-02-233-4/+75
| |/ / / | | | | | | | | | | | | Closes #360.
* | | | Merge branch 'debug-datastream' into 'main'eta2022-02-243-2/+19
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | impl Debug for DataStream (and its components) See merge request tpo/core/arti!351
| * | | impl Debug for DataStream (and its components)Ian Jackson2022-02-243-2/+19
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | My proximate motivation is that tls-api wants its inner streams to be Debug. But in general, I agree with the Rust API Guidelines notion that almost everything should be Debug. I have gone for the "dump all the things" approach. A more nuanced approach would be possible too.
* | | Merge branch 'dir-store-redux' into 'main'Nick Mathewson2022-02-246-297/+350
|\ \ \ | | | | | | | | | | | | | | | | Dir store redux See merge request tpo/core/arti!345
| * | | dirmgr::Store.expire_all takes configtharvik2022-02-233-26/+47
| | | |
| * | | dirmgr: add Store traittharvik2022-02-236-278/+310
| | | |
* | | | Fix typosDimitris Apostolou2022-02-231-3/+3
|/ / /
* | | Merge branch 'custom-tcp-hook' into 'main'eta2022-02-232-0/+344
|\ \ \ | |_|/ |/| | | | | | | | add example of tcp hook See merge request tpo/core/arti!341
| * | add one more comment to hook-tcptrinity-1686a2022-02-231-0/+1
| | |
| * | examples/hook-tcp: add some comments, rework lifetimes a biteta2022-02-231-28/+73
| | | | | | | | | | | | | | | | | | Try to make the `hook-tcp` example a bit easier to read by adding/changing comments, and renaming the lifetimes for `async_trait`-generated trait methods.
| * | implement correct handling of connection close and add comment explaining ↵trinity-1686a2022-02-231-9/+54
| | | | | | | | | | | | goal of the example
| * | add drop on CustomTcpStream to catch disconnection without closetrinity-1686a2022-02-231-0/+13
| | |
| * | add example of tcp hooktrinity-1686a2022-02-232-0/+240
| | | | | | | | | | | | this required to make additional types public
* | | Merge remote-tracking branch 'origin/mr/340'Nick Mathewson2022-02-2313-121/+117
|\ \ \
| * | | Cleanups on !340 from @dizietIan Jackson2022-02-231-2/+8
| | | |
| * | | Make NoLock into BadApiUsage.Nick Mathewson2022-02-224-23/+15
| | | | | | | | | | | | | | | | | | | | | | | | To implement this, we had to refactor the tor_circmgr api for flushing state changes to disk, so that it checks if it has the lock, and only then tries to store.
| * | | Eliminate RequestedResourceAbsent kind.Nick Mathewson2022-02-226-30/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was only one use of this, and it was in as-yet-unused relay-only code. Removing this type required refactoring the relay onion handshake code to use its own error type, which is probably clever anyway.
| * | | Fold NamespaceFull into BadApiUsage.Nick Mathewson2022-02-222-13/+1
| | | |
| * | | Rename RemoteNameError to RemoteHostNotFoundNick Mathewson2022-02-222-6/+6
| | | | | | | | | | | | | | | | | | | | This is a bit ugly but we need it to work around the problem where exits aren't always clear about _why_ a hostname lookup failed.
| * | | Replace TorNetworkError with TorDirectoryErrorNick Mathewson2022-02-222-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 TorAccessFailedNick Mathewson2022-02-223-4/+4
| | | |
| * | | 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-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