Opened 3 years ago

Closed 3 years ago

#19881 closed defect (implemented)

New guard plan - guard selection for circuits

Reported by: andrea Owned by: nickm
Priority: Medium Milestone: Tor: 0.3.0.x-final
Component: Core Tor/Tor Version: Tor: unspecified
Severity: Normal Keywords: isaremoved, nickwants029, tor-guards-revamp, nickm-deferred-20161005, TorCoreTeam201611
Cc: asn Actual Points:
Parent ID: #19877 Points: 5
Reviewer: Sponsor: SponsorU-must

Description

New guard plan summarized at https://lists.torproject.org/pipermail/tor-dev/2016-July/011234.html

5) Selecting guards for circuits

  • Meant to replace choose_random_entry_impl()
  • See section SELECTING in prop271
  • Add new circuit states to or_circuit_t
  • Implement the guard selection logic
  • Unittests on circuit state machine
  • Unittests on guard selection logic

Child Tickets

Change History (8)

comment:1 Changed 3 years ago by isabela

Keywords: isaremoved added
Milestone: Tor: 0.2.9.x-finalTor: 0.2.???

comment:2 Changed 3 years ago by nickm

Keywords: nickwants029 tor-guards-revamp added

Batch modification: I believe these are all (or nearly all!) required for the successful completion of the SponorU guards task.

comment:3 Changed 3 years ago by asn

Cc: asn added

comment:4 Changed 3 years ago by isabela

Milestone: Tor: 0.2.???Tor: 0.2.9.x-final
Sponsor: SponsorUSponsorU-must

comment:5 Changed 3 years ago by nickm

Keywords: nickm-deferred-20161005 added
Milestone: Tor: 0.2.9.x-finalTor: 0.3.0.x-final

Deferring more things -- even ones I love -- to 0.3.0. Please argue if I'm wrong.

comment:6 Changed 3 years ago by andrea

Owner: set to nickm
Status: newassigned

Reassigning some prop 271 tickets to nickm for parallel work in Oct.

comment:7 Changed 3 years ago by nickm

These features are now implemented in my (much-rebased, very unstable) prop271 branch. Because #19889 is not done yet, however, they don't actually do anything yet. At least there are tests and documentation?

comment:8 Changed 3 years ago by nickm

Keywords: TorCoreTeam201611 added
Resolution: implemented
Status: assignedclosed
Note: See TracTickets for help on using tickets.