-
Notifications
You must be signed in to change notification settings - Fork 725
-
Notifications
You must be signed in to change notification settings - Fork 725
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature] Firewall live log improvements #1788
Comments
@AdSchellevis for CI it would be better if the table would match the rest of the pages (only the play button is green). Also the (i) could be a button or removed if the line is clickable. For future reasons: |
I like it. :) Name with parenthesis looks a bit odd, „Firewall Live Log“ sounds best.
… On 4. Nov 2017, at 05:22, Fabian Franz, BSc. ***@***.***> wrote:
@AdSchellevis for CI it would be better if the table would match the rest of the pages (only the play button is green). Also the (i) could be a button or removed if the line is clickable.
For future reasons:
Array fields in the model have an UUID which can be used in the firewall and the view can restore the original values. This way the displayed value would always match.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
It has some rough edges, but it's almost finished. "Firewall Live Log" sounds good, I will replace the caption. The colouring might need some work, probably a bit too bright at the moment, the different table styling is intentional (our default tables are a bit to large for this purpose). The uuid's are indeed practical, but since not stored in pf itself, they won't help us here (but we could consider changing the filterlog part to capture some more info which isn't available at kernel level).... Not for now, maybe later. |
Wow, great work! I like it already. |
…ange color scheme and fix some JS issues for #1788
We still need cleanups to remove the old dynamic view and related legacy code, but the functionality should work now. In case anybody want to test the new functionality, use the following commands to replace the old dynamic view for the new live firewall log view:
(I had some issues pulling log_helper in via opnsense-patch, hence the curl) |
…use the same service call. for #1788
…e can easily map list of results. #1788
All done, thanks! |
Having just upgraded my first OPNsense box I just wanted to say thank you. I love the new live log ;) |
Up to now I always used Sophos UTM (free edition) as firewall/router VMs, because they were one of the first to natively support Hyper-V. I am now switching to OPNsense.
One thing I miss is Sophos UTM's live firewall log (see screenshot). I know there is the Dynamic View in OPNsense, but IMHO UTM's version is much more clearly laid out (although the look is a bit old-fashioned) and functional in order to see what's going on.
What I would like to see in OPNsense:
The text was updated successfully, but these errors were encountered: