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

Re: [tor-bugs] #22217 [Metrics/metrics-lib]: Parse new padding-counts lines



#22217: Parse new padding-counts lines
---------------------------------+-----------------------------------
 Reporter:  karsten              |          Owner:  karsten
     Type:  enhancement          |         Status:  needs_review
 Priority:  Medium               |      Milestone:  metrics-lib 1.7.0
Component:  Metrics/metrics-lib  |        Version:
 Severity:  Normal               |     Resolution:
 Keywords:                       |  Actual Points:
Parent ID:                       |         Points:
 Reviewer:                       |        Sponsor:
---------------------------------+-----------------------------------

Comment (by iwakeh):

 Please find four commits in [https://gitweb.torproject.org/user/iwakeh
 /metrics-lib.git/?h=task-22217 my repo branch].

 Open topics I noticed in other parts of the code:
 The empty key issue seems to also apply to comma separated key-value pairs
 and maybe others.  Similarly repeated keys.  New ticket?

 It would be good to add the check for the correct exception string also to
 other test methods in ExtraInfoDescriptorTest.  Another ticket?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/22217#comment:4>
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