[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #21759 [Metrics/CollecTor]: Add persistence for torperf/onionperf
#21759: Add persistence for torperf/onionperf
-------------------------------+---------------------------------
Reporter: iwakeh | Owner: iwakeh
Type: enhancement | Status: needs_revision
Priority: Medium | Milestone: CollecTor 1.3.0
Component: Metrics/CollecTor | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points:
Parent ID: | Points:
Reviewer: | Sponsor:
-------------------------------+---------------------------------
Comment (by iwakeh):
Common approach for storing:
It turns out that the functionality in OnionperfDownloader was too
restrictive.
Here's an overview:
- bridge-desc (all types): after sanitation the descriptor is written; if
one descriptor cannot be sanitized, it is skipped
- relay-desc (all types): descriptors written one by one skipping
problematic ones
- exitlists: always stored as a single file.
Solution: if a single TorperfResult is not parseable, it should simply be
skipped as in the sync-implementation.
Another topic, which relates to comment:7:
Currently, all synced descriptors receive their annotation from the
Annotation enum (cf. package o.t.c.conf.Annotation). This happens,
because only the raw bytes are taken from a given descriptor and written
to the file system.
But, actually the annotation(s) should be taken from the 'getAnnotations'
method and be prepended to the raw descriptor bytes, shouldn't they?
The fix for this would also simplify some sync-code. New ticket also for
milestone 1.3.0?
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21759#comment:10>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs