[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #15989 [Tor]: Add AccountingRule in and out
#15989: Add AccountingRule in and out
----------------------------------+------------------------------------
Reporter: teor | Owner:
Type: enhancement | Status: needs_revision
Priority: Low | Milestone: Tor: 0.2.8.x-final
Component: Tor | Version: Tor: unspecified
Severity: Normal | Resolution:
Keywords: lorax tor-relay easy | Actual Points:
Parent ID: | Points:
Sponsor: |
----------------------------------+------------------------------------
Changes (by teor):
* status: needs_review => needs_revision
Comment:
(I'm awake and working and not in the middle of something!)
Replying to [comment:9 unixninja92]:
> In {{{log_accounting}}} for SUM we shouldn't have to double
{{{acc_bytes}}}, right?
Yeah, I have no idea why that was implemented. Let's get rid of it.
> Other changes are pushed.
Thanks! Looks great.
One more issue:
In options_validate, the text should mention in and out:
{{{
REJECT("AccountingRule must be 'sum' or 'max'");
}}}
Once those two changes are done, I think this is good to go.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/15989#comment:10>
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