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

Re: [tor-bugs] #5482 [BridgeDB]: Annotate and sort the social-network bridge pools by stability and reachability

#5482: Annotate and sort the social-network bridge pools by stability and
reachability
------------------------------+---------------------------------------------
 Reporter:  karsten           |          Owner:  aagbsn        
     Type:  project           |         Status:  needs_revision
 Priority:  normal            |      Milestone:                
Component:  BridgeDB          |        Version:                
 Keywords:  SponsorF20121101  |         Parent:                
   Points:                    |   Actualpoints:                
------------------------------+---------------------------------------------
Changes (by karsten):

  * status:  needs_review => needs_revision


Comment:

 I finished reviewing 2/3 of the #5482 code.  I'm afraid there's a major
 difference between the #4255 code and this code that makes BridgeDB do
 something vastly different than evaluated in the #4255 analysis: the #5482
 code looks at bridge ''server descriptors'' for calculating stability
 metrics, not bridge ''network statuses''.  But bridge server descriptors
 are only published every 12--18 hours, in contrast to bridge network
 statuses that are published every 30 minutes.  Results will vary a lot. --
 Do you think you can change the #5482 code to use information from bridge
 network statuses instead?  There's nothing in bridge server descriptors
 that we even care about for the stability metrics described in #4255.

 A few minor remarks:
  - `db.getBridgeHistory(self.wmtbac).*` in `Bridges.py` should pass
 `self.fingerprint` as parameter, not `self.wmtbac`.
  - `"%Y-%M-%d %H:%m:%S"` has `%M` and `%m` interchanged; see my last
 comment above.
  - `weighting_factor = long(19)/long(20)` is always 0.
  - In `Stability.py` there are misspelled references to `weighedTime` and
 `allWeighedTimes`.
  - `"DELETE * FROM BridgeDescriptors WHERE timestamp > ?"` purges
 descriptors younger than a given timestamp, not older as the comment says.

 Thanks!

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/5482#comment:8>
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