ClearOS Bug Tracker


View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000011ClearOSdeprecated - ClearOS 5 Protocol Filterpublic2010-01-18 16:472010-07-15 12:01
ReporterVejlefjordskolen 
Assigned Todsokoloski 
PrioritynormalSeveritytweakReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version5.1 
Target Version5.2Fixed in Version5.2 
Summary0000011: Protocol Filter report does not show number of packets or bytes over 100,000
DescriptionIn the API, /var/webconfig/api/Layer7Filter.class.php, you match only digits in the output from iptables where the number of packets and bytes are shown, but when there are more than 100,000 packets or bytes, the number also contains letters, e.g. 100K instead of 100000. This causes your matching to fail, resetting the counters to zero.
TagsNo tags attached.
Attached Filestgz file icon counter-fix.tgz [^] (979 bytes) 2010-04-18 15:58

- Relationships

-  Notes
(0000080)
dsokoloski (developer)
2010-04-15 16:13

Source Code Changelog
---------------------------------------------------
- Fix for packet/byte counters with M/K/G postfix. [fixed tracker 0000011]

File Changes
---------------------------------------------------
U legacy/webconfig/trunk/api/Layer7Filter.class.php
(0000081)
Vejlefjordskolen (reporter)
2010-04-18 15:58
edited on: 2010-04-18 16:03

The fix checked in will not work correctly, as the protocol-filter-report pages do not support strings in the fields for the counters, only numbers.

I have attached my fix as patch sets from the current SVN trunk. (I hope I did it right, I've never made patch sets before). It has an extra case for handling the possibility of no counter, as the JavaScript on the protocol-filter-report page does not support empty strings in the XML.

(0000082)
dsokoloski (developer)
2010-04-18 16:09

Source Code Changelog
---------------------------------------------------
- Changed layer-7 protocol format string from decimal to string. [fixed tracker 0000011]

File Changes
---------------------------------------------------
U legacy/webconfig/trunk/htdocs/admin/protocol-filter-report.php
(0000083)
dsokoloski (developer)
2010-04-18 16:23

Source Code Changelog
---------------------------------------------------
- Changed layer-7 protocol format string from decimal to string. [fixed tracker 0000011]

File Changes
---------------------------------------------------
U legacy/webconfig/trunk/htdocs/admin/protocol-filter-report.xml.php

- Issue History
Date Modified Username Field Change
2010-01-18 16:47 Vejlefjordskolen New Issue
2010-01-19 08:16 user2 Severity minor => tweak
2010-01-19 08:16 user2 Status new => acknowledged
2010-01-19 08:16 user2 Target Version => 5.2
2010-03-25 20:29 user2 Status acknowledged => assigned
2010-03-25 20:29 user2 Assigned To => dsokoloski
2010-04-15 16:13 dsokoloski Checkin
2010-04-15 16:13 dsokoloski Note Added: 0000080
2010-04-15 16:13 dsokoloski Status assigned => resolved
2010-04-15 16:13 dsokoloski Resolution open => fixed
2010-04-18 15:58 Vejlefjordskolen Note Added: 0000081
2010-04-18 15:58 Vejlefjordskolen Status resolved => feedback
2010-04-18 15:58 Vejlefjordskolen Resolution fixed => reopened
2010-04-18 15:58 Vejlefjordskolen File Added: counter-fix.tgz
2010-04-18 16:03 Vejlefjordskolen Note Edited: 0000081
2010-04-18 16:09 dsokoloski Checkin
2010-04-18 16:09 dsokoloski Note Added: 0000082
2010-04-18 16:09 dsokoloski Status feedback => resolved
2010-04-18 16:09 dsokoloski Resolution reopened => fixed
2010-04-18 16:23 dsokoloski Checkin
2010-04-18 16:23 dsokoloski Note Added: 0000083
2010-05-19 11:30 user2 Fixed in Version => 5.2
2010-07-15 12:01 user2 Status resolved => closed
2018-01-15 16:31 user2 Category app-protocol-filter - Protocol Filter Manager => app-protocol-filter - ClearOS 5
2018-01-15 16:32 user2 Category app-protocol-filter - ClearOS 5 => deprecated - ClearOS 5 Protocol Filter