fmadio / public

fmadio issue tracking
MIT License
8 stars 3 forks source link

Packet Scope Filter list missing #313

Closed fmadio closed 4 years ago

fmadio commented 5 years ago

on all ROMs the default packetscope filter list is missing. need to update with some basic and common filters

fmadio commented 4 years ago

default filter list would be

FCS Errors : frame.fcs == fail TCP Only : bpf == tcp UDP only : bpf == udp DNS only : bpf == udp port 53 port 0 : frame.port == 0 port 1 : frame.port == 1 . . port 7

dlannan-fmad commented 4 years ago

Will add the same filters from the file split list. Ahh whoops - updated page and there it is :)

dlannan-fmad commented 4 years ago

Is a drop down suitable? Which will effectively create a "starting" block filter with one of the above set?

fmadio commented 4 years ago

yup drop down, its setup here, but not working

image

fmadio commented 4 years ago

which is loaded by the file /opt/fmadio/etc/packetscope_filter.lua

feel free to change anything, its all broken currently

fmadio commented 4 years ago

hmm some issues tho lol

image

dlannan-fmad commented 4 years ago

Ok. I think I have a working version. Fixed a couple of little weird issues. Its still not entirely right.

dlannan-fmad commented 4 years ago

Changed the script.js include to a module. This was a good move there were a number of bugs/issues resolved:

dlannan-fmad commented 4 years ago

Another note:

dlannan-fmad commented 4 years ago

Aaron - just checking. Let me know if the module type is a problem. The key problem it may cause is compatibility with older browsers (anything about 4yrs or older)

dlannan-fmad commented 4 years ago

Odd issue:

dlannan-fmad commented 4 years ago

The odd issue:

What I added, is when you select a "default filter" then it calls the delete on each row (to ensure the internal list is correct). This appears to work, with the exception of frame.port == fail -- Weird.

dlannan-fmad commented 4 years ago

Ok. I think I have a working version in my dave branch. Will do some more testing to be sure, before putting into staging.

fmadio commented 4 years ago

Excellent, wait for the staging PR

fmadio commented 4 years ago

checked in staging works well, very cool

pushed to master closing