Skip to content
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

IDS/IPS Eventforwarding #17

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

IDS/IPS Eventforwarding #17

wants to merge 7 commits into from

Conversation

majst01
Copy link
Contributor

@majst01 majst01 commented Jun 13, 2021

  • enable forwarding of different event types

@GrigoriyMikhalkin

TODO:

  • implement IDS and IPS forwarding, test basic funcitonality in mini-lab
  • ensure suricata can send IDS events to unix_dgram or unix_stream for eve output, decide which dgram|stream to use

TODO firewall-controller:

  • move suricate configuration reconcilation from metal-networker to firewall-controller
  • enable/disable IDS logforwarding depending on Firewall CRD

TODO after all above:

  • same for IPS

@majst01 majst01 changed the title droptailer migrated IDS/IPS Eventforwarding Jun 13, 2021
@GrigoriyMikhalkin
Copy link

GrigoriyMikhalkin commented Jun 25, 2021

Regarding socket type. https://redmine.openinfosecfoundation.org/issues/250#note-12 -- it looks like there's not much of a difference between using dgrams and stream, since log size usually fits in single packet payload.

Also, i tested forwarding from dgram socket and it works(with some modification to the code).

@GrigoriyMikhalkin
Copy link

GrigoriyMikhalkin commented Jul 2, 2021

@majst01 I'm not sure if it's worth to have separate types for IDS/IPS events. In both cases, logs are written to eve socket. Only difference is in event(action) types. IMO, it's probably better(simpler) to leave only single type(IDS) for both cases.

@majst01
Copy link
Contributor Author

majst01 commented Jul 4, 2021

Yes sure, we should forward both events into the same stream.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants