[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #6363 [Tor Directory Authority]: Make directory authorities vote on "a" lines in consensus network status documents

#6363: Make directory authorities vote on "a" lines in consensus network status
documents
-------------------------------------+--------------------------------------
 Reporter:  ln5                      |          Owner:  ln5           
     Type:  enhancement              |         Status:  needs_revision
 Priority:  normal                   |      Milestone:                
Component:  Tor Directory Authority  |        Version:                
 Keywords:  SponsorG20120930         |         Parent:  #4564         
   Points:                           |   Actualpoints:                
-------------------------------------+--------------------------------------
Changes (by nickm):

  * status:  accepted => needs_revision


Comment:

 Replying to [comment:6 ln5]:
 > Replying to [comment:5 nickm]:
 > > method 13 needs to become method 14 -- 13 just got used by bug6404.
 >
 > OK, that's now been fixed in a7716707, same branch.
 >
 > > Will need microdesc support before it's done.
 >
 > Yes.
 >
 > > Does this need to be deployed and producing "a" lines for September?
 If so, it needs to go in 0.2.3.x, unfortunately.  ("No" is the answer that
 would make me happie.)
 >
 > External (SponsorG) deadline is September 30. Why does it have to go
 > in 0.2.3? Wouldn't the 0.2.4.1 (around September 7) be a good release
 > for this?

 Because the authorities won't all upgrade to 0.2.4.x on that timeframe.

 > > The logic for setting the 3rd argument for compare_vote_rs is pretty
 complicated! Some comments might help me out.
 >
 > I'm afraid I don't understand. Do you mean that the places where we're
 > calling compare_vote_rs() are hard to understand? The
 >
 >     consensus_method >= MIN_METHOD_FOR_A_LINES
 >
 > in compute_routerstatus_consensus()?
 >

 yes, those.

 > > The new memset() in networkstatus_compute_consensus() is redundant
 with #6514, already fixed.
 >
 > Fixed in e6225a33, same branch.

 thanks

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/6363#comment:7>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs