Page MenuHomePhabricator

doc: tor.md updates
ClosedPublic

Authored by PiRK on Feb 10 2022, 13:15.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABCbb0557c2cc37: doc: tor.md updates
Summary

core#20757:

doc: update -proxy, -onion and -onlynet info in tor.md

Improve the description of what these options do with regards to
tor or network traffic.

doc: update/improve automatic tor section of tor.md

doc: update tor.md manual config, move after automatic config

core#20587:

[doc] Tidy up Tor doc (more stringent)

This is a backport of core#20757 [2/2] (all the tor.md changes from that PR) and core#20587

Depends on D11031

Test Plan

proofreading

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

PiRK requested review of this revision.Feb 10 2022, 13:15
Fabien requested changes to this revision.Feb 10 2022, 15:45
Fabien added a subscriber: Fabien.
Fabien added inline comments.
doc/tor.md
59 ↗(On Diff #32330)

I think you can keep this

200 ↗(On Diff #32330)

This is from another PR ?

This revision now requires changes to proceed.Feb 10 2022, 15:45
PiRK retitled this revision from doc: tor.md updates (20757) to doc: tor.md updates.
PiRK edited the summary of this revision. (Show Details)

I accidentaly included changes from two other PRs. Mention core#20587 in the summary (fully backported in this diff), revert the -natpmp change from core#18077 which depends on other commits from that PR

Fabien requested changes to this revision.Feb 11 2022, 21:13

Then you're missing some of the changes from PR20587

This revision now requires changes to proceed.Feb 11 2022, 21:13

add the missing change from PR20587 ("Hidden" -> "Onion")

This revision is now accepted and ready to land.Feb 14 2022, 09:11
This revision was automatically updated to reflect the committed changes.