[metrics-bugs] #27390 [Metrics/CollecTor]: Properly clean up sanitized web server logs in the recent/ directory

[Tor Bug Tracker & Wiki]

#27390: Properly clean up sanitized web server logs in the recent/ directory
-------------------------------+------------------------------
 Reporter:  karsten            |          Owner:  metrics-team
     Type:  defect             |         Status:  merge_ready
 Priority:  Medium             |      Milestone:
Component:  Metrics/CollecTor  |        Version:
 Severity:  Normal             |     Resolution:
 Keywords:                     |  Actual Points:
Parent ID:                     |         Points:
 Reviewer:  irl                |        Sponsor:
-------------------------------+------------------------------
Changes (by irl):

 * status:  needs_review => merge_ready


Comment:

 How much does the new function in PersistenceUtils differ from the similar
 functions:

 {{{
 onionperf/OnionPerfDownloader.java:  public void cleanUpRsyncDirectory()
 throws ConfigurationException {
 exitlists/ExitListDownloader.java:  public void cleanUpRsyncDirectory() {
 bridgedescs/SanitizedBridgesWriter.java:  public void
 cleanUpRsyncDirectory() throws ConfigurationException {
 relaydescs/ArchiveWriter.java:  public void cleanUpRsyncDirectory() {
 }}}

 Currently webstats does seem to be the only module that uses the shared
 function. The relaydescs module may need something more complex to handle
 persisting microdescriptors for longer, but the other modules could maybe
 just switch? This could be a new ticket.

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