[tor-bugs] #6708 [pyonionoo]: Pyonionoo returns code 500 for a few parameters

[Tor Bug Tracker & Wiki]

#6708: Pyonionoo returns code 500 for a few parameters
-----------------------+----------------------------------------------------
 Reporter:  karsten    |          Owner:  gsathya          
     Type:  defect     |         Status:  needs_information
 Priority:  normal     |      Milestone:                   
Component:  pyonionoo  |        Version:                   
 Keywords:             |         Parent:                   
   Points:             |   Actualpoints:                   
-----------------------+----------------------------------------------------
Changes (by gsathya):

  * status:  needs_review => needs_information


Comment:

 Replying to [comment:13 karsten]:
 > Replying to [comment:12 gsathya]:
 > > What exactly is the bug here? Can you tell me how to reproduce it?
 >
 > The bug is that the parser doesn't handle bridges without any flags
 correctly.  An example line is:
 >
 > {{{
 > b Unnamed 1D36547129EF6399E487ED359B32AA7ECBCD43FC 10.157.98.209;;
 2012-09-18 01:37:03 443 0  -1 ?? null -1 null null
 > }}}
 >
 > Note that two space characters between 0 and -1.  That's where the
 missing flags would go.
 Aha gotcha! I'm wondering why not have a default value of '??' or "null"
 if there are no flags in the summary document? It makes sense to not use
 whitespace as a value since we're parsing it based on position of the
 text.

 Merged the other two commits. Thanks!

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