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

test: add test for bug 6109 - v1 #1229

Closed
wants to merge 1 commit into from

Conversation

jufajardini
Copy link
Contributor

In IDS mode, with midstream disabled, it seems that the exception policy 'reject' is applying actions that should only be taken in IPS mode. This leads to the flow being dropped (or logged as such?) in IDS mode. This test showcases this.

Bug #6109 (https://redmine.openinfosecfoundation.org/issues/6109)

This test fails, currently.

In IDS mode, with midstream disabled, it seems that the exception policy
'reject' is applying actions that should only be taken in IPS mode. This
leads to the flow being dropped (or logged as such?) in IDS mode. This
test showcases this.

Bug #6109
@jufajardini jufajardini added bug Something isn't working requires suricata fix This PR requires an issue in Suricata to be fixed first labels Jun 5, 2023
@catenacyber catenacyber added tests pass These new tests should pass and removed bug Something isn't working requires suricata fix This PR requires an issue in Suricata to be fixed first labels Jun 20, 2023
@catenacyber
Copy link
Collaborator

Suricata PR was merged, and now these tests pass locally for me.
@jufajardini would you rebase and force-push to get CI running and checking all is green ?

@jasonish jasonish mentioned this pull request Jun 20, 2023
@jasonish
Copy link
Member

Merged via #1263. Thanks!

@jasonish jasonish closed this Jun 20, 2023
@jufajardini jufajardini deleted the sv-6109/v1 branch June 21, 2023 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests pass These new tests should pass
3 participants