Opened 2 years ago

Closed 10 months ago

#19704 closed enhancement (implemented)

Deprecate ReachableDirAddresses and ClientPreferIPv6DirPort

Reported by: teor Owned by: nickm
Priority: Medium Milestone: Tor: 0.3.2.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: 030-proposed
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description (last modified by teor)

In 0.2.8, we made all client directory fetches use begindir over the ORPort.

This makes ReachableDirAddresses and ClientPreferIPv6DirPort useless for clients. And they don't apply to relays, so we should get rid of them.

We should also check and remove all the reachability code that uses FIREWALL_DIR_CONNECTION. This will help to simplify that code as well.

Child Tickets

Change History (10)

comment:1 Changed 2 years ago by teor

Description: modified (diff)
Summary: Deprecate ReachableDirAddresses, and remove code that uses itDeprecate ReachableDirAddresses and ClientPreferIPv6DirPort

comment:2 Changed 21 months ago by teor

Milestone: Tor: 0.2.???Tor: 0.3.???

Milestone renamed

comment:3 Changed 20 months ago by teor

We should also remove the DirPort text:
"Setting up a DirPort on IPv6 is useful if clients can't use an ORPort."

comment:4 in reply to:  3 Changed 20 months ago by teor

Replying to teor:

We should also remove the DirPort text:
"Setting up a DirPort on IPv6 is useful if clients can't use an ORPort."

That was https://trac.torproject.org/projects/tor/wiki/doc/IPv6RelayHowto
fixed now.

comment:5 Changed 19 months ago by nickm

Keywords: tor-03-unspecified-201612 added
Milestone: Tor: 0.3.???Tor: unspecified

Finally admitting that 0.3.??? was a euphemism for Tor: unspecified all along.

comment:6 Changed 14 months ago by arma

Milestone: Tor: unspecifiedTor: 0.3.2.x-final

Putting it in milestone 0.3.2 because that's where #20575 is.

comment:7 Changed 14 months ago by nickm

Keywords: tor-03-unspecified-201612 removed

Remove an old triaging keyword.

comment:8 Changed 10 months ago by nickm

Owner: set to nickm
Status: newaccepted

trivial; done in branch ticket19704

comment:9 Changed 10 months ago by nickm

Status: acceptedneeds_review

comment:10 Changed 10 months ago by nickm

Resolution: implemented
Status: needs_reviewclosed

Merged

Note: See TracTickets for help on using tickets.