[metrics-bugs] #22208 [Metrics/metrics-lib]: Provide lines containing extra, unrecognized arguments
Tor Bug Tracker & Wiki
blackhole at torproject.org
Tue May 9 15:13:49 UTC 2017
#22208: Provide lines containing extra, unrecognized arguments
-------------------------------------+--------------------------
Reporter: karsten | Owner: metrics-team
Type: enhancement | Status: new
Priority: Medium | Milestone:
Component: Metrics/metrics-lib | Version:
Severity: Normal | Keywords:
Actual Points: | Parent ID:
Points: | Reviewer:
Sponsor: |
-------------------------------------+--------------------------
This question came up when implementing
[https://trac.torproject.org/projects/tor/ticket/21934#comment:4 #21934]:
do we need to include lines with extra arguments in
`Descriptor#getUnrecognizedLines()`? Or do we need another method
`Descriptor#getPartiallyUnrecognizedLines()`? Or can we safely drop those
extra arguments on the floor? See also
`TorperfResults#getUnrecognizedKeys()` which is somewhat related here, but
not exactly the same.
This is not super urgent, because at least the bytes of the entire
descriptor are available for further processing.
The fix is also not trivial to write, because we'll have manually go
through all places where extra arguments are permitted and ignored and add
those lines to a list of partially unrecognized lines.
The result could be that doing nothing is the way to go. Needs more
thoughts.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/22208>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the metrics-bugs
mailing list