Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#20960 closed defect (implemented)

Extend sybil-detection to authorities as well

Reported by: nickm Owned by: nickm
Priority: Medium Milestone: Tor: 0.3.0.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords:
Cc: arma Actual Points: .1
Parent ID: Points:
Reviewer: Sponsor:


In get_possible_sybil_list(), we exempt addresses shared with directory authorities from sybil detection. This served a purpose long ago, when Roger had to run extra relays on the moria host all the time (for debugging purposes), but I hope we're past those days.

Child Tickets

Change History (6)

comment:1 Changed 4 years ago by nickm

Owner: set to nickm
Status: newaccepted

Roger says he's fine with this.

comment:2 Changed 4 years ago by nickm

Actual Points: .1
Status: acceptedneeds_review

ticket20960 in my public repository removes AuthDirMaxServersPerAuthAddr to implement this change.

comment:3 Changed 4 years ago by arma

Status: needs_reviewmerge_ready

Looks good!

(I didn't try compiling, so be sure to do that before merging. :)

comment:4 Changed 4 years ago by nickm

Resolution: implemented
Status: merge_readyclosed


comment:5 Changed 4 years ago by teor

(This might affect the tor test network, I have emailed the list.)

comment:6 Changed 4 years ago by nickm

If it affects the test network, would it suffice to raise the limit for _all_ IPs on the test network, with the non-removed "AuthDirMaxServersPerAddr" option?

Note: See TracTickets for help on using tickets.