[metrics-bugs] #22983 [Metrics/metrics-lib]: add a descriptor interface and implementation for web-logs
Tor Bug Tracker & Wiki
blackhole at torproject.org
Fri Jul 28 08:35:43 UTC 2017
#22983: add a descriptor interface and implementation for web-logs
---------------------------------+-----------------------------------
Reporter: iwakeh | Owner: metrics-team
Type: enhancement | Status: needs_review
Priority: Medium | Milestone: metrics-lib 2.1.0
Component: Metrics/metrics-lib | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points:
Parent ID: | Points:
Reviewer: | Sponsor:
---------------------------------+-----------------------------------
Comment (by karsten):
I need more time for this review. But here's a first question:
Should we really put the sanitizing code into metrics-lib rather than
CollecTor? That's an important design decision and a change to what we
have been doing in the past. Where would this code be used other than in
CollecTor? So far, metrics-lib has primarily been the client-side library
for applications using CollecTor data. But this change would turn it into
a library that both the CollecTor server and its clients depend on. I'm
yet undecided whether this is a good idea or not. In any case, we should
discuss this first.
I'll continue my review later today and post my findings here.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/22983#comment:10>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the metrics-bugs
mailing list