Page MenuHomePhabricator
Differential D11032

doc: tor.md updates
ClosedPublic
Actions

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
Branch
pr20685
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 18237
Build 36282: Build Diff
Build 36281: arc lint + arc unit

Event Timeline

PiRK created this revision.Feb 10 2022, 13:15
Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 10 2022, 13:15
PiRK requested review of this revision.Feb 10 2022, 13:15
teamcity edited the summary of this revision. (Show Details)Feb 10 2022, 13:15
Harbormaster completed remote builds in B18237: Diff 32330.Feb 10 2022, 13:15
PiRK edited the summary of this revision. (Show Details)Feb 10 2022, 13:16
PiRK added a parent revision: D11031: doc: tor.md updates (19961).
Fabien requested changes to this revision.Feb 10 2022, 15:45
Fabien added a subscriber: Fabien.
Fabien added inline comments.
doc/tor.md
59

I think you can keep this

200

This is from another PR ?

This revision now requires changes to proceed.Feb 10 2022, 15:45
PiRK updated this revision to Diff 32351.Feb 11 2022, 15:09
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

Harbormaster completed remote builds in B18253: Diff 32351.Feb 11 2022, 15:09
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
PiRK updated this revision to Diff 32358.Feb 14 2022, 07:39

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

Harbormaster completed remote builds in B18260: Diff 32358.Feb 14 2022, 07:40
Fabien accepted this revision.Feb 14 2022, 09:11
This revision is now accepted and ready to land.Feb 14 2022, 09:11
Closed by commit rABCbb0557c2cc37: doc: tor.md updates (authored by saibato <saibato.naga@pm.me>, committed by PiRK). · Explain WhyFeb 15 2022, 18:09
This revision was automatically updated to reflect the committed changes.
PiRK added a commit: rABCbb0557c2cc37: doc: tor.md updates.

Revision Contents
Changeset List

PathSize
doc/
159 lines

Diff 32330

View Options

doc/tor.md

Loading...