Opened 15 months ago

Closed 5 weeks 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 15 months ago by teor

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

comment:2 Changed 11 months ago by teor

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

Milestone renamed

comment:3 Changed 10 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 10 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 10 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 5 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 5 months ago by nickm

Keywords: tor-03-unspecified-201612 removed

Remove an old triaging keyword.

comment:8 Changed 6 weeks ago by nickm

Owner: set to nickm
Status: newaccepted

trivial; done in branch ticket19704

comment:9 Changed 6 weeks ago by nickm

Status: acceptedneeds_review

comment:10 Changed 5 weeks ago by nickm

Resolution: implemented
Status: needs_reviewclosed

Merged

Note: See TracTickets for help on using tickets.