[tor-bugs] #20412 [Metrics/Onionoo]: Skip bad archived descriptors rather than aborting the entire import

Tor Bug Tracker & Wiki blackhole at torproject.org
Tue Nov 7 21:25:32 UTC 2017


#20412: Skip bad archived descriptors rather than aborting the entire import
-----------------------------+------------------------------
 Reporter:  karsten          |          Owner:  metrics-team
     Type:  defect           |         Status:  merge_ready
 Priority:  Medium           |      Milestone:
Component:  Metrics/Onionoo  |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:  metrics-2017     |  Actual Points:
Parent ID:  #20548           |         Points:
 Reviewer:  iwakeh           |        Sponsor:
-----------------------------+------------------------------

Comment (by iwakeh):

 Replying to [comment:9 karsten]:
 > Replying to [comment:8 iwakeh]:
 > > The fix looks ok and ready for merge.
 >
 > Will do! Thanks for checking!
 >
 > > (Was the initial implementation just chance or was there a reason for
 halting the import?)
 >
 > Hmm? Do you mean the initial ''import''? If so, I guess it worked out,
 because we only started including unparseable descriptors in CollecTor's
 output a few years ago.

 True.

 >
 > > I'm wondering, if the actual issue isn't in metrics-lib.
 > > Shouldn't there be a method `finishReading` that halts the reading
 process on client request?  Clients shouldn't be forced to read all
 available descriptors.  Afaik, the ISE is just due to calling the
 statistics methods before all was read.  New ticket for metrics-lib?
 >
 > Yes, that's another issue that we should fix. To be clear, this fix here
 is also a valid one: we shouldn't stop the import just because we ran into
 a single unparseable descriptor.

 Yes, my question above was an aside; not directed at the solution in this
 ticket.

 >
 > But what you point out is another issue: we shouldn't be forced to read
 all available descriptors. I'm not certain what the best fix is there.
 Maybe a `finishReading` or `abortReading` might work. But that's something
 that we can discuss on the new ticket. Can you open one? Thanks!

 See #24166.

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


More information about the tor-bugs mailing list