net: Remove "tor" as a network specification

[fanquake]

"tor" as a network specification was deprecated in 60dc8e4 in favor of "onion"
and this commit removes it and updates the relevant test.

Previously #16029. This has been warning as being deprecated since v0.17.0.

This PR only removes the already deprecated usage of tor as a network specification, the use of tor throughout the codebase, is not deprecated.

[DrahtBot]

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/34031.

Reviews

See the guideline for information on the review process.

Type	Reviewers
ACK	laanwj, davidgumberg, janb84, stickies-v
Stale ACK	pablomartin4btc, sipa

If your review is incorrectly listed, please copy-paste <!--meta-tag:bot-skip--> into the comment that the bot should ignore.

[pablomartin4btc]

ACK 7efb18c

[sipa]

ACK 7efb18c

14 major releases ought to be a long enough deprecation period.

[fanquake]

cc @laanwj

[laanwj]

There's one left in init.cpp:

diff --git a/src/init.cpp b/src/init.cpp
index bfb9483ad8730541848fc4789bafe343165d28f2..9849be22f9e45387cf1303b05100cdee58496f90 100644
--- a/src/init.cpp
+++ b/src/init.cpp
@@ -1683,7 +1683,7 @@ bool AppInitMain(NodeContext& node, interfaces::BlockAndHeaderTipInfo* tip_info)
             ipv4_proxy = name_proxy = proxy;
         } else if (net_str == "ipv6") {
             ipv6_proxy = name_proxy = proxy;
-        } else if (net_str == "tor" || net_str == "onion") {
+        } else if (net_str == "onion") {
             onion_proxy = proxy;
         } else if (net_str == "cjdns") {
             cjdns_proxy = proxy;

[laanwj]

Test needs a change too:

diff --git a/test/functional/feature_proxy.py b/test/functional/feature_proxy.py
index ba8a0212a6a0dd9cf166ab87d3b60486cb844aba..aec81fa79888af42dec3387e4a2ccddee0d5204c 100755
--- a/test/functional/feature_proxy.py
+++ b/test/functional/feature_proxy.py
@@ -468,8 +468,8 @@ class ProxyTest(BitcoinTestFramework):
         assert_equal(nets["ipv6"]["proxy"], "127.6.6.6:6666")
         self.stop_node(1)

-        self.log.info("Test overriding the Tor proxy")
-        self.start_node(1, extra_args=["-proxy=127.1.1.1:1111", "-proxy=127.2.2.2:2222=tor"])
+        self.log.info("Test overriding the Onion proxy")
+        self.start_node(1, extra_args=["-proxy=127.1.1.1:1111", "-proxy=127.2.2.2:2222=onion"])
         nets = networks_dict(self.nodes[1].getnetworkinfo())
         assert_equal(nets["ipv4"]["proxy"], "127.1.1.1:1111")
         assert_equal(nets["ipv6"]["proxy"], "127.1.1.1:1111")

[laanwj]

Code review ACK e7ac5a1

[davidgumberg]

crACK e7ac5a1

[janb84]

ACK e7ac5a1

As a review I did some searches for tor usage and there is still a lot of usage of the name tor. This is not an issue but what is not super clear to me is to what extend tor is deprecated or where we would like to use onion now.

suggestion NIT: add some clarity in the PR description that only the tor network specification is deprecated and not the usage of the name tor or code referring to tor (or something similar) to limit followup PR's that see this as an invitation to remove the name Tor everywhere.

[stickies-v]

ACK e7ac5a1

[laanwj]

suggestion NIT: add some clarity in the PR description that only the tor network specification is deprecated and not the usage of the name tor or code referring to tor (or something similar) to limit followup PR's that see this as an invitation to remove the name Tor everywhere.

Right. Usage of "Tor" is fine, and doesn't need to be removed. It's just that Tor is a general overlay network with exis nodes, but this specific proxy is the one used for .onion addresses, so calling it onion is way less confusing.