[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-bugs] #22208 [Metrics/metrics-lib]: Provide lines containing extra, unrecognized arguments



#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
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs