Opened 4 weeks ago

Closed 4 weeks ago

Last modified 12 days ago

#33091 closed defect (fixed)

Remove redundant checks in ip_address_changed()

Reported by: teor Owned by: teor
Priority: Medium Milestone: Tor: 0.4.3.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: ipv6, tor-relay, technical-debt
Cc: Actual Points: 0.1
Parent ID: #33049 Points: 0.1
Reviewer: Sponsor: Sponsor55-can


ip_address_changed() does a separate exit policy check, but all relays need to refresh their descriptors when their address changes.

Child Tickets

Change History (4)

comment:1 Changed 4 weeks ago by teor

Cc: babut massar rl1987@… sven.herzberg@… removed
Keywords: technical-debt added
Status: assignedneeds_review

See my PR:

If the reviewer thinks it's simple enough, this change can go in 0.4.3.

comment:2 Changed 4 weeks ago by dgoulet

lgtm; Merged to master!

comment:3 Changed 4 weeks ago by dgoulet

Milestone: Tor: 0.4.4.x-finalTor: 0.4.3.x-final
Resolution: fixed
Status: needs_reviewclosed

comment:4 Changed 12 days ago by teor

Parent ID: #5940#33049
Note: See TracTickets for help on using tickets.