-
Notifications
You must be signed in to change notification settings - Fork 100
Mime type reload #2606
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
base: master
Are you sure you want to change the base?
Mime type reload #2606
Conversation
@@ -0,0 +1,19 @@ | |||
requires: | |||
features: | |||
- HAVE_MIMETYPE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jasonish seeing green CI for this highlights that we need SV coverage ;-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please elaborate? What would this look like?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You seemed to have more ideas than me.
I was just saying that manual runs should be done for releases to ensure that every SV test/check is run.
You were saying that there were tools to do that ;-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No tools that I know of. I think Victor put out a wish directed my way that tools exist, to which I had no comment :)
count: 1 | ||
match: | ||
event_type: alert | ||
alert.signature_id: 1338 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: we could check that the alert event has the mimetype
Add tests for new mime type computation and match.
Redmine ticket: https://redmine.openinfosecfoundation.org/issues/7816