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

flow: Add tests for excluding pkt recursion from flow #2081

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

coledishington
Copy link
Contributor

Add tests for verifying matching packet flows when including and excluding pkt recursion from flow matching.

Bug: #6260

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

@catenacyber
Copy link
Collaborator

Requires OISF/suricata#10214

Comment on lines 12 to 15
The terminated tests are checking when the suricata is an inline IPS
device that is terminating a tunnel. Both flow directions are tested,
first packet ingress on non-tunneled interface and first packet ingress
on tunneled interface.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to see if I got this right, are the terminated tests the ips-tunnel-pkt-flows ones?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct. I missed updating the README.md. Force-pushed the changes as it was a docs update (rather than code change).

Add tests for verifying matching packet flows when including and
excluding pkt recursion from flow matching.

Bug: #6260
@coledishington coledishington force-pushed the flow-optional-recur-limit-6260-v6 branch from 5bea92e to 65bb9bb Compare October 16, 2024 18:56
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.

3 participants