Snort Widget fix (Snort 2.9.2.3 pkg v. 2.5.0)

Cino

thanks digdug3!! Confirm it works for me on pfSense 2.1

_igor_

works on pfSensre 2.0.1 too. Great work! thx

marcelloc

Open a pull request for this fix on https://github.com/bsdperimeter/pfsense-packages.

This way ermal can check the code and commit.

digdug3

Thanks, but I think the code is not clean enough. Maybe I'll clean it later on. Besides that, the code is mostly a copy-paste…

digdug3

Just cleaned the code more and removed a bug.
Please test. If it works correctly maybe ermal can commit it.

_igor_

works.

p0intman

Thank you so much I have been waiting to use this widget forever.

_igor_

Therer seems to be a problem: I have 4 instances of the snort widget on my dashboard. If i delete one, all are deleted.

digdug3

@_igor_:

Therer seems to be a problem: I have 4 instances of the snort widget on my dashboard. If i delete one, all are deleted.

How did you get 4 instances? I can't get more than one running. Besides, why would you want that?
Can you get more instances of the Captive Portal or Firewall Logs as well?
Do you have the widescreen package installed?

_igor_

hey, you understood wrong - i dont want to have 4 instances. I activate one and get 4. Deleting one of them deletes all. There is an error.

Yes, i have widescreen installed. I dont have captive portal.
Activated firewall logs and have only one instance. It has to do with the snort alert widget, i think.

digdug3

Ah, yes, I misunderstood you!

Can't replicate that, but I know there are some problems with the Widescreen package.
Is it possible for you to remove the widescreen package and see if the problem still exists?

mschiek01

@digdug3:

Ah, yes, I misunderstood you!

Can't replicate that, but I know there are some problems with the Widescreen package.
Is it possible for you to remove the widescreen package and see if the problem still exists?

I have widescreen and four interfaces although snort is only running on 2 of them I have not had any problems as of yet.

digdug3

@ermal thanks for committing the code

Just viewed your changes, but with your code the widget will first display all alerts from IF 0 then IF 1 and so on,
I think it is more desireable to display alerts sorted by date, not by interface. Thats why I added the sorting of timestamps.