https://redmine.pfsense.org/https://redmine.pfsense.org/favicon.ico?16780521162017-02-27T09:34:20ZpfSense bugtrackerpfSense - Bug #7323: More user friendly defaults for firewall logs viewhttps://redmine.pfsense.org/issues/7323?journal_id=318282017-02-27T09:34:20ZJim Pingle
<ul></ul><p>I had initially insisted that option default to off because it was horribly slow on ALIX and other low-end platforms at the time it was introduced, but most of those have gone EOL or won't be supported on 2.4 anyhow. Also the formatting left some things to be desired on 2.2.x.</p>
<p>On 2.4 the output being in another column seems to be the best-looking option, and it doesn't seem to be too bad on SG-1000 speed-wise at the default 50 lines of output. Also using a second row seems to be a tad slower.</p>
<p>Some envelope style calculations for 10 iterations at 50 (default) and 500 lines of firewall log data on an SG-1000, which is for all intents and purposes the minimum specs for 2.4:</p>
<p><img src="http://i.imgur.com/kTydUav.png" alt="" /></p>
<pre>
Setting Lines Avg Time (sec)
No Descr 50 1.673
Descr Col 50 1.918
Descr Row 50 1.935
No Descr 500 6.099
Descr Col 500 6.435
Descr Row 500 6.514
</pre>
<p>So overall it only adds 1/4 to 1/2 sec on average to load them in a second column on the slowest hardware we have on hand that runs 2.4. The 50 line case takes the bigger hit at a 14.64% increase vs 5.51% increase on 500 lines, but either way it's probably worth it now to default it on for new configs.</p> pfSense - Bug #7323: More user friendly defaults for firewall logs viewhttps://redmine.pfsense.org/issues/7323?journal_id=318292017-02-27T09:40:12ZJim Pingle
<ul></ul><p>It is worth noting that the above numbers were with a minimal ruleset, it would be worth repeating the test with a more complicated ruleset that had a wider variety of log data to work with. The speed result was what was displayed by using Firebug's net panel while clicking the tab name to refresh the log (F5 caused it to refresh more page assets/content rather than only reloading the page)</p> pfSense - Bug #7323: More user friendly defaults for firewall logs viewhttps://redmine.pfsense.org/issues/7323?journal_id=318302017-02-27T10:28:34ZKill Bill
<ul></ul><p>Nice timings. ;) Just for a giggle, I tried with Alix and 2.3.3: the times are ~ 6.5s with 50 lines and ~25s with 500 lines (with the description column view and pretty small ruleset, using the Google Chrome timeline.)</p> pfSense - Bug #7323: More user friendly defaults for firewall logs viewhttps://redmine.pfsense.org/issues/7323?journal_id=318512017-02-28T12:30:06ZJim Pingle
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Feedback</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Applied in changeset <a class="changeset" title="In a stock/default config.xml, show firewall rule descriptions in a column. Fixes #7323 Previous ..." href="https://redmine.pfsense.org/projects/pfsense/repository/2/revisions/88aceb2573acab8fbc7e4d467e31280f701ba9cd">88aceb2573acab8fbc7e4d467e31280f701ba9cd</a>.</p> pfSense - Bug #7323: More user friendly defaults for firewall logs viewhttps://redmine.pfsense.org/issues/7323?journal_id=318522017-02-28T12:37:11ZJim Pingle
<ul><li><strong>Assignee</strong> set to <i>Jim Pingle</i></li></ul><p>I changed the default value in the stock config.xml rather than trying to do any sort of complicated shuffling of config parameter interpretation.</p>
<p>This way new users will get the column and existing users have their preference preserved.</p>
<p>If we decide that we want to turn it on for everyone, then we could add upgrade code to set <code>$config['syslog']['filterdescriptions'] = 1</code> so we don't have to change the way the setting is interpreted or stored.</p> pfSense - Bug #7323: More user friendly defaults for firewall logs viewhttps://redmine.pfsense.org/issues/7323?journal_id=318992017-03-02T09:38:32ZJim Pingle
<ul><li><strong>Status</strong> changed from <i>Feedback</i> to <i>Resolved</i></li></ul><p>A fresh install has the column active in the firewall log as expected.</p> pfSense - Bug #7323: More user friendly defaults for firewall logs viewhttps://redmine.pfsense.org/issues/7323?journal_id=319152017-03-02T11:26:14ZJohn Murphy
<ul></ul><p>Just verified as well, but a little late to the plate it appears. fresh install of CE from 20170302(<a class="external" href="https://snapshots.pfsense.org/amd64/pfSense_master/installer/pfSense-CE-2.4.0-BETA-amd64-20170302-0727.iso.gz">https://snapshots.pfsense.org/amd64/pfSense_master/installer/pfSense-CE-2.4.0-BETA-amd64-20170302-0727.iso.gz</a>) within a VM has default selection of "Display as Column". Existing install that was updated to 2.4 (on SG-2220) preserved pre-existing selection for the drop down in question.</p>