[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