warvair / peerblock

Automatically exported from code.google.com/p/peerblock
Other
0 stars 1 forks source link

Default "show allowed connections" option to false #89

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Due to the performance impact of logging large amounts of traffic, we
should consider defaulting the "show allowed connections" option to false.
 This way new users shouldn't come away with the opinion that PeerBlock is
a performance pig, chewing up an ungodly amount of their CPU.

The only problem with implementing this is that it might become
questionable as to whether PeerBlock is actually doing anything, if you
don't see anything being logged.  We might want to log the first few
allowed-packets even if "show allowed connections" is off, then post a
message to that log-window that we're disabling logging of additional
allowed-packets for performance reasons or something.

Original issue reported on code.google.com by peerbloc...@gmail.com on 12 Sep 2009 at 3:15

GoogleCodeExporter commented 9 years ago
Okay, so in addition to defaulting ShowAllowed to false for fresh installs, what
we're doing from here on in is to display the first 10 allowed packets 
regardless of
ShowAllowed so that the user doesn't think we're broken.  We then display a log
message (with the text in the Range column; you'll need to hover your mouse 
over that
field to see the full message) explaining why we're stopping.

Original comment by peerbloc...@gmail.com on 13 Sep 2009 at 3:32

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r148.

Original comment by peerbloc...@gmail.com on 13 Sep 2009 at 3:33