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 Passed
Unit
No Test Coverage
Build Status
Buildable 18260
Build 36328: Build Diff
Build 36327: arc lint + arc unit

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.