Skip to content

Conversation

regit
Copy link
Contributor

@regit regit commented Jul 19, 2025

Add tests for new mime type computation and match.

Redmine ticket: https://redmine.openinfosecfoundation.org/issues/7816

@regit regit mentioned this pull request Jul 19, 2025
5 tasks
@@ -0,0 +1,19 @@
requires:
features:
- HAVE_MIMETYPE
Copy link
Collaborator

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 ;-)

Copy link
Member

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?

Copy link
Collaborator

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 ;-)

Copy link
Member

@jasonish jasonish Jul 22, 2025

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
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires suricata pr Depends on a PR in Suricata
Development

Successfully merging this pull request may close these issues.

4 participants