search

darold / squidanalyzer

Squid Analyzer parses Squid proxy access log and reports general statistics about hits, bytes, users, networks, top URLs, and top second level domains. Statistic reports are oriented toward user and bandwidth control.
http://squidanalyzer.darold.net/
126 stars 36 forks source link

Througput title says B/s but value is TransferUnits/s #134

Closed aabaker closed 8 years ago

aabaker commented 8 years ago

This seems to occur consistently through the program. I'm not sure if it is the intention throughput is always in bytes per second (which would make some sense as throughput numbers will generally be a lot smaller than other transfer statistics) in which case I guess $network_stat{$network}{bytes} et al need scaling up by $self->{TransfertUnitValue} when used in a throughput calculation or if it is intended that throughput matches the scale used elsewhere which gets marginally tricky as B/s in the throuput header is outside the Translate{} whereas $self->{TransfertUnit} does get translated.

darold commented 8 years ago

Las commit b25874a might solves this issue. The throughput unit label will now be changed automatically following the formatting unit. Thanks for the report.