#25523: Add support for webstats tarballs
-----------------------------+-----------------------------------
 Reporter:  karsten          |          Owner:  iwakeh
     Type:  defect           |         Status:  needs_revision
 Priority:  Medium           |      Milestone:  metrics-lib 2.3.0
Component:  Metrics/Library  |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:                   |  Actual Points:
Parent ID:                   |         Points:
 Reviewer:                   |        Sponsor:
-----------------------------+-----------------------------------
Changes (by karsten):

 * status:  needs_review => needs_revision


Comment:

 This patch does enable metrics-lib to parse webstats tarballs. So far so
 good!

 However, `Descriptor#getDescriptorFile` now returns a "virtual" `File`
 reference for the tarball entry, rather than the `File` reference of the
 tarball. For example, here are the results of that method for a webstats
 log and consensus:

 {{{
 www-staging.torproject.org_stellatum.torproject.org_access.log_20150211.xz
 in/consensuses-2017-02.tar.xz
 }}}

 The specification of that method says: "Return the file, tarball or plain
 file, that contained this descriptor, or null if this descriptor was not
 read from a file."

 Would you like to provide a fix for that? And can you include a change log
 entry for the change? Thanks!

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/25523#comment:4>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Reply via email to