[tor-bugs] #2286 [Tor Directory Authority]: We still use self-published relay bandwidth sometimes

[Tor Bug Tracker & Wiki]

#2286: We still use self-published relay bandwidth sometimes
-------------------------------------+--------------------------------------
 Reporter:  arma                     |          Owner:                    
     Type:  defect                   |         Status:  needs_revision    
 Priority:  major                    |      Milestone:  Tor: 0.2.3.x-final
Component:  Tor Directory Authority  |        Version:                    
 Keywords:  arma-cares               |         Parent:                    
   Points:                           |   Actualpoints:                    
-------------------------------------+--------------------------------------
Changes (by nickm):

  * status:  needs_review => needs_revision


Comment:

 Replying to [comment:33 arma]:
 > Looks like it should work. Three thoughts:
 >
 > A) I notice that this plan doesn't affect flags at all. So we could have
 a relay with the Fast flag but a weight of 20 (which would ordinarily not
 warrant the Fast flag). That's a separate topic (whether authorities
 should vote flags based on the descriptor bandwidth, the vote bandwidth,
 or now the consensus bandwidth), but I'd be more comfortable if we raise
 our default cap to e.g. 100.

 Okay, I should do this.

  [...]
 > C)
 > {{{
 > -    "w" SP "Bandwidth=" INT [SP "Measured=" INT] NL
 > +    "w" SP "Bandwidth=" INT [SP "Capped=1"] [SP "Measured=" INT] NL
 > }}}
 > Is the order mandatory here? Sure looks like it is. I guess in our
 current implementation it doesn't matter, since we never say Capped and
 Measured together.

 Order doesn't have to be mandatory in votes, I guess.  Might as well leave
 it that way though.

 > Smaller thoughts:
 >
 > A) run make check-spaces on it

 will do

 > B) the spec change says "maxunmappedbw" which is probably not the right
 word.

 should probably say "uncapped".


 WRT the other matters -- arma says that there need to be corresponding
 changes to bwauth before we can merge this, and that he'll amke the
 tickets for that.

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/2286#comment:36>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online