[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #30196 [Core Tor/sbws]: Add the tor version to the sbws bandwidth file header
#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
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs