summaryrefslogtreecommitdiff
path: root/crates/arti/src
Commit message (Collapse)AuthorAgeFilesLines
* Fix typosDimitris Apostolou2024-01-081-1/+1
|
* arti: Fix log message typo.Gabriela Moldovan2023-12-141-1/+1
|
* Merge branch 'return-of-the-son-of-arti-todos' into 'main'Nick Mathewson2023-12-144-59/+140
|\ | | | | | | | | Resolve Arti crate todos, redux See merge request tpo/core/arti!1833
| * Trivial case fix in a comment.Ian Jackson2023-12-141-1/+1
| |
| * arti: Add a todo to describe how and why to test build_list.Nick Mathewson2023-12-131-0/+7
| |
| * arti: Use BTreeMap for maps of onion service configs.Nick Mathewson2023-12-131-9/+9
| |
| * arti: Simplify build_list slightlyNick Mathewson2023-12-131-4/+3
| |
| * arti: Use a macro to declare modules as conditionally public.Nick Mathewson2023-12-131-25/+36
| | | | | | | | | | (We can't use `visibility::make(pub)` or `visible` here. Try it yourself if you don't believe me!)
| * arti: Test for expected value of onion service cfg.Nick Mathewson2023-12-121-2/+34
| |
| * arti: resolve a TODO in onion_proxy launch_new.Nick Mathewson2023-12-121-7/+2
| |
| * arti: rethink a TODO HSS about nicknamesNick Mathewson2023-12-121-1/+9
| | | | | | | | | | | | The TODO HSS in question is about our inability to serialize every possible builder. The right answer here might be to use something else instead of a ListBuilder.
| * arti: Enforce onion service nickname distinctnessNick Mathewson2023-12-122-18/+39
| | | | | | | | | | | | | | | | | | | | | | The most logical way to do this was to change the "List" type to a HashMap, and add a build function to the ListBuilder. This change additionally renames: OnionServiceProxyConfig{List=>Map} NamedProxyMap => ProxyBuilderMap (We now have two kinds of map, and this name change will clarify the distinction.)
| * arti: Downgrade a TODO HSS about testing onion-service config.Nick Mathewson2023-12-121-1/+2
| | | | | | | | | | | | | | IIUC, there will never be a Some(InNew) entry here, since we will never have an onion service be configured by default. Instead we test this kind of configuration by having commented-out options that we uncomment as needed.
| * arti: Generalize a TODO HSS about reconfiguration.Nick Mathewson2023-12-122-1/+10
| | | | | | | | | | | | It's correct that we'd like someday for the `arti` crate APIs to allow all the different modes supported by `Reconfigure` enum; this is #1156, and it does not block an HSS release.
| * arti: conditionally expose onion_proxy module when experimental-api is enabledNick Mathewson2023-12-121-4/+4
| |
| * Remove TODOs obsoleted by onion service reconfiguration.Nick Mathewson2023-12-122-2/+0
| | | | | | | | (This was solved with !1798)
* | arti: Print out the onion address retrieved from StateMgr.Gabriela Moldovan2023-12-141-1/+15
| |
* | arti: Add an hss subcommand.Gabriela Moldovan2023-12-141-2/+46
| | | | | | | | Part of #1071
* | arti: Use warn_report on anyhow::Error in onion_proxyNick Mathewson2023-12-121-4/+1
|/ | | | This was made possible by !1818.
* Merge branch 'todo_hsrproxy' into 'main'Nick Mathewson2023-12-111-1/+4
|\ | | | | | | | | Resolve several TODO HSS items in hsrproxy See merge request tpo/core/arti!1809
| * hsrproxy: Take nickname as an argument and pass it to forwarding functionNick Mathewson2023-12-071-1/+4
| | | | | | | | We'll use this for logging.
* | Add a TODO HSS comment about Reconfigure.Ian Jackson2023-12-111-0/+1
| |
* | Reconfigure onion services when their configuration changes.Nick Mathewson2023-12-052-28/+20
| | | | | | | | Closes #1089.
* | arti: Make reconfiguration slightly more abstractNick Mathewson2023-12-052-24/+94
| | | | | | | | | | | | | | | | | | | | Now instead of having a hardwired list of of things to reconfigure, the watch_cfg module now has a vector of ReconfigurableModule. As noted in the documentation, I don't intend that this should be our final API here: It is deliberately not exposed. When we revisit the structure of `arti` more, we should probably do this differently.
* | arti: Backend support for reconfiguring onion proxies.Nick Mathewson2023-12-051-6/+101
|/
* Merge branch 'msrv-followup' into 'main'Nick Mathewson2023-12-041-16/+4
|\ | | | | | | | | Followups from MSRV 1.70 upgrade See merge request tpo/core/arti!1785
| * arti: rewrite ilog10_roundup using checked_ilog10.Nick Mathewson2023-11-301-16/+4
| |
* | Merge branch 'dirs' into 'main'gabi-2502023-12-041-2/+0
|\ \ | | | | | | | | | | | | Cleanup and clarification re cache_dir and state_dir (rename in tor_dirmgr) See merge request tpo/core/arti!1789
| * | arti-example-config: Do not claim that state_dir is unusedIan Jackson2023-12-041-2/+0
| | | | | | | | | | | | tor-ptmgr uses it, and the key manager too.
* | | arti cfg: suppress a warningIan Jackson2023-12-041-0/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | There is a TODO HSS for this. Fixes cargo check --workspace --all-targets producing warning: unused variable: `result` --> crates/arti/src/cfg.rs:1060:13 | 1060 | let result = file.resolve::<(TorClientConfig, ArtiConfig)>(); | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_result` | = note: `#[warn(unused_variables)]` on by default
* / Move from `trust-dns` to `hickory`Andrew2023-11-301-6/+6
|/ | | | | | This is not a change of ownership but just rebranding. Signed-off-by: Andrew <[email protected]>
* Revert "hsrproxy: Temporarily change the format of ProxyRule."Nick Mathewson2023-11-292-9/+6
| | | | | | | | | | | This reverts commit 21605d2c9e601c3a5099bfd8d8c887cbb3b36c0a. We're doing this because we wanted the configuration of proxy rules to be a 2-tuple, as in `[ "22", "127.0.0.1:22" ]`. But `config` couldn't handle that before, so we temporarily changed it to a string, as in "22 => 127.0.0.1:22". Closes #1058.
* Bump to clap 4 and resolve compiler errors.Nick Mathewson2023-11-151-2/+2
|
* handle_socks_conn: update parameter name in doc commentJim Newsome2023-10-251-1/+1
|
* HSS: Do not drop ProxySet immediately - let HS services continue to run (fmt)Ian Jackson2023-10-191-2/+4
|
* HSS: Do not drop ProxySet immediately - let HS services continue to runIan Jackson2023-10-191-4/+19
| | | | Dropping the ProxySet causes all the services to terminate.
* HSS: Add many must_use annotationsIan Jackson2023-10-191-0/+2
|
* Merge branch 'generic-listen' into 'main'Nick Mathewson2023-10-163-55/+70
|\ | | | | | | | | arti, tor-config: Allow listening on generic addresses for SOCKS and DNS. See merge request tpo/core/arti!1613
| * Treat only EAFNOSUPPORT as a warningJani Monoses2023-09-262-8/+12
| |
| * Do not writelns in Display, undo localhost_port_legacy changes.Jani Monoses2023-09-251-1/+1
| |
| * Handle address already in useJani Monoses2023-09-222-2/+6
| |
| * arti, tor-config: Allow listening on generic addresses for SOCKS and DNS.Jani Monoses2023-09-223-56/+63
| |
* | Merge branch 'configure_onion_service_part2' into 'main'Nick Mathewson2023-10-112-29/+103
|\ \ | | | | | | | | | | | | Non-working attempt to test onion service configuration. See merge request tpo/core/arti!1640
| * | Use a less obtrustive sign to end the onion services config sectionNick Mathewson2023-10-112-7/+5
| | |
| * | Improve test fn documentation even more.Ian Jackson2023-10-101-2/+2
| | |
| * | arti::cfg: tweak narrow() test fn to be more useful.Nick Mathewson2023-10-102-13/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | It still matches the start pattern in the same way, but it takes the _next_ match of the end pattern after the start pattern, and it allows the end pattern to be absent. This will make the bridge tests a little more robust, and let the onion service tests stop saying BEGIN and END.
| * | hsrproxy: Temporarily change the format of ProxyRule.Nick Mathewson2023-10-101-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | Since config-rs has a bug breaking its support for nested arrays (see discussion at !1640), we're temporarily changing the format to a single string with a "=>" in the middle. I've opened #1058 to put things back as they were once config-rs is fixed.
| * | Non-working attempt to test onion service configuration.Nick Mathewson2023-10-032-22/+76
| | | | | | | | | | | | | | | | | | | | | The problem here is that, for some reason, the proxy_rule configuration doesn't actually work. I suspect a bug in Flatten, since the same configuration works fine if you pass it to the hsrproxy config directly.
| * | arti: Try to document some test helpers.Nick Mathewson2023-10-031-1/+18
| | |
* | | arti: Lower some over-broad #allows in onion_proxy.rsNick Mathewson2023-10-051-4/+2
| | |