Clarify "address" in man page: IPv4, IPv6, hostname?
The DirAuthority line only takes an IPv4 address as an "address". But other torrc options take IPv6 addresses or hostnames.
We should clarify what we mean when we say "address".
Reported by Andrew Smith: https://lists.torproject.org/pipermail/tor-relays/2017-February/011876.html
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- teor changed milestone to %Tor: 0.3.2.x-final
changed milestone to %Tor: 0.3.2.x-final
Trac:
Milestone: Tor: unspecified to Tor: 0.3.1.x-finalDeferring all 0.3.1 tickets with status == new, owner == nobody, sponsor == nobody, points > 0.5, and priority < high.
I'd still take patches for most of these -- there's just nobody currently lined up to work on them in this timeframe.
Trac:
Keywords: doc easy deleted, doc, triaged-out-20170308, easy added
Milestone: Tor: 0.3.1.x-final to Tor: unspecifiedTrac:
Username: kleft
Cc: N/A to kleft
Parent: N/A to #18892 (moved)Trac:
Keywords: doc, triaged-out-20170308, easy deleted, tor-doc easy intro manpage addedTrac:
Milestone: Tor: unspecified to Tor: 0.3.2.x-final
Owner: N/A to nickm
Status: new to acceptedmy branch
ticket21405
has tweaks here.Trac:
Status: accepted to needs_reviewIn the case of
__IP__
, I don't see any mentions of what is the expected format? I'm asking because for v4 it is easy but seems that for v6 we do require[]
from what I can see in the code in terms of parsing.It's kind of obvious the brackets are needed for IPv6 because you can put a port with
:PORT
but could we make a quick note maybe at the start of the man page that every IPv6 address needs the brackets?Trac:
Status: needs_review to needs_revisionI've thrown some boilerplate around whenever we say
__IP__
. Better now?Hmmm I don't see anything for
ReachableAddresses
,ReachableDirAddresses
andReachableORAddresses
?I think they all document that they're in the same format as ExitPolicy? Also ReachableDirAddresses is deprecated.
Oh yes! Good stuff.
Trac:
Status: needs_revision to merge_readyok; merged!
Trac:
Status: merge_ready to closed
Resolution: N/A to implemented- Trac closed
closed
- Trac changed time estimate to 8h
changed time estimate to 8h
- Trac moved to tpo/core/tor#21405 (closed)
moved to tpo/core/tor#21405 (closed)