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

Tor Bug Tracker & Wiki torproject-admin at torproject.org
Tue Sep 25 01:50:44 UTC 2012


#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:11 karsten]:
 > Replying to [comment:9 gsathya]:
 > > There's a problem with f503b794a, the database schema looks for a
 integer consensus_weight and gets a "None" when values[9] == '??'.  We
 should probably set them to a default value(onionoo sets it to '-1L')
 since this is a required parameter(I think None breaks the sql query as
 well).
 >
 > Oops.  I looked closer at the bug I was trying to fix in f503b79, and it
 seems it was caused by bridges without any flags, not by relays/bridges
 with consensus_weight "??".  I took out f503b79 and added e006843 for that
 problem.
 >
 > Please review my force-updated
 [https://gitweb.torproject.org/user/karsten/pyonionoo.git/shortlog/refs/heads/bug_6708_revised
 bug_6708_revised branch] again?

 What exactly is the bug here? Can you tell me how to reproduce it? Also,
 you're using the {{{split()}}} method but the commit message talks about
 {{{strip()}}} method, you might want to change that.

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


More information about the tor-bugs mailing list