[tor-bugs] #30196 [Core Tor/sbws]: Add the tor version to the sbws bandwidth file header

Tor Bug Tracker & Wiki blackhole at torproject.org
Thu Feb 13 13:31:09 UTC 2020


#30196: Add the tor version to the sbws bandwidth file header
---------------------------+-----------------------------------
 Reporter:  teor           |          Owner:  (none)
     Type:  enhancement    |         Status:  merge_ready
 Priority:  High           |      Milestone:  sbws: 1.2.x-final
Component:  Core Tor/sbws  |        Version:
 Severity:  Major          |     Resolution:
 Keywords:  sbws-roadmap   |  Actual Points:
Parent ID:  #33121         |         Points:  1
 Reviewer:  ahf            |        Sponsor:
---------------------------+-----------------------------------

Comment (by juga):

 Replying to [comment:16 ahf]:
 > I think both the spec change and the sbws changes are good, but I have
 two questions:
 >
 > 1. What is the purpose of the `xxx`'s with 'tech-dept'? Is the goal we
 go back here and do something actionable and is that something that is
 best to have in the code rather than in tickets?

 If i'd create a ticket with a tech-debt (i guess i wrote a typo there)
 changes, i don't know if it'll ever be solved and then in the ticket would
 need to point to all the parts of the code where i detected it while
 working on something else.

 > 2. The constant renaming seems OK to me, but it seems complicated to
 maintain all these lists and how they are subsets/supersets of each other?

 agree, do you have a suggestion on how to change that without having to
 increase minor version because changing API?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/30196#comment:17>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list