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

[Tor Bug Tracker & Wiki]

#30196: Add the tor version to the sbws bandwidth file header
---------------------------+-----------------------------------
 Reporter:  teor           |          Owner:  (none)
     Type:  enhancement    |         Status:  needs_revision
 Priority:  Medium         |      Milestone:  sbws: 1.2.x-final
Component:  Core Tor/sbws  |        Version:
 Severity:  Normal         |     Resolution:
 Keywords:                 |  Actual Points:
Parent ID:  #30255         |         Points:  1
 Reviewer:                 |        Sponsor:
---------------------------+-----------------------------------
Changes (by teor):

 * status:  new => needs_revision


Comment:

 Replying to [comment:6 juga]:
 > Reminder to myself: i've implemented the operating system, ssl_version
 and tor_version in branch https://github.com/juga0/sbws/tree/ticket30196

 Here's my review of tor_version:
 * the code looks good
 * please refactor EXTRA_ARG_KEYVALUES and BANDWIDTH_HEADER_KEY_VALUES_INIT
 so you're not adding tor_version in two different places
 * please submit a bandwidth-file-spec patch, so the reviewer knows what
 your code should do
 * does the "chg: v3bwfile: !refactor, obtain headers from state" commit
 belong in a separate pull request? It's quite large.

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