[metrics-bugs] #24823 [Metrics/Website]: Avoid logging an exception when the client aborts a CSV file download

Tor Bug Tracker & Wiki blackhole at torproject.org
Tue Jan 9 20:23:20 UTC 2018


#24823: Avoid logging an exception when the client aborts a CSV file download
-----------------------------+--------------------------------
 Reporter:  karsten          |          Owner:  metrics-team
     Type:  defect           |         Status:  needs_revision
 Priority:  Low              |      Milestone:
Component:  Metrics/Website  |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:                   |  Actual Points:
Parent ID:                   |         Points:
 Reviewer:                   |        Sponsor:
-----------------------------+--------------------------------
Changes (by iwakeh):

 * status:  needs_review => needs_revision


Comment:

 Replying to [comment:2 karsten]:
 > Hmm, or maybe we should not attempt to send an error code after sending
 a (partial) response to the client. Please review
 [https://gitweb.torproject.org/karsten/metrics-
 web.git/commit/?h=task-24823&id=f310c0dab2a30aa6f7b7a844e4304e6f347168bb
 commit f310c0d in my task-24823 branch]. (Untested!)

 This is a good idea. Maybe, the ticket title should be changed to 'Avoid
 sending an error after a (partial) response.'?

 I would want to keep the try-with-resources in method 'writeStatsFile' in
 order to close everything properly;  the exceptions don't need to be
 caught in there.  Setting to needs-revision.

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


More information about the metrics-bugs mailing list