Opened 5 months ago

Closed 5 months ago

Last modified 5 months 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:

Description

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 5 months ago by nickm

  • Owner set to nickm
  • Status changed from new to accepted

Roger says he's fine with this.

comment:2 Changed 5 months ago by nickm

  • Actual Points set to .1
  • Status changed from accepted to needs_review

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

comment:3 Changed 5 months ago by arma

  • Status changed from needs_review to merge_ready

Looks good!

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

comment:4 Changed 5 months ago by nickm

  • Resolution set to implemented
  • Status changed from merge_ready to closed

merged!

comment:5 Changed 5 months ago by teor

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

comment:6 Changed 5 months 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.