Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Detect origin keyword 6794 v2 #11770

Closed

Conversation

scottfgjordan
Copy link
Contributor

Make sure these boxes are checked accordingly before submitting your Pull Request -- thank you.

Contribution style:

Our Contribution agreements:

Changes (if applicable):

Link to ticket: https://redmine.openinfosecfoundation.org/issues/6794

Describe changes:

  • Applied fixes from initial review
  • Introduced a destination keyword as the counterpart to origin (was suggested to be target, but that keyword is already taken)
  • Added copy device to a live device to enable the above.
  • Added some documentation describing interface role

Provide values to any of the below to override the defaults.

Add option of role to live device configuration. Possible
roles are : trusted, untrusted, and unknown. Configurable
via suricata.yaml. AF-Packet and DPDK runmodes supported,
others default to unknown role.
Allows for matching against packets based on the role of the
live device. The origin of a packet refers to the role of the
live device where the flow originated from.
Adds the copy device to the live device structure. Valid only
in IPS mode.
Allows for matching against packets based on the role of the
copy live device. For IPS only. The destination of a packet
refers to the role of the copy live device.
Copy link

NOTE: This PR may contain new authors.

Copy link

codecov bot commented Sep 12, 2024

Codecov Report

Attention: Patch coverage is 90.59829% with 44 lines in your changes missing coverage. Please review.

Project coverage is 82.54%. Comparing base (31bed10) to head (bfedb1d).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11770      +/-   ##
==========================================
- Coverage   82.62%   82.54%   -0.08%     
==========================================
  Files         919      921       +2     
  Lines      248979   249432     +453     
==========================================
+ Hits       205722   205903     +181     
- Misses      43257    43529     +272     
Flag Coverage Δ
fuzzcorpus 60.28% <13.46%> (-0.59%) ⬇️
livemode 18.72% <35.89%> (+0.01%) ⬆️
pcap 44.09% <13.46%> (-0.04%) ⬇️
suricata-verify 61.84% <13.46%> (-0.05%) ⬇️
unittests 59.06% <83.76%> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@scottfgjordan
Copy link
Contributor Author

Whoops, commit author got a bit messed up (they are my github username instead of my name). I will rebase and amend in the next version of PR.

For the fuzzer failures, from what I can tell they're unrelated to the changes in the PR?

@catenacyber
Copy link
Contributor

For the fuzzer failures, from what I can tell they're unrelated to the changes in the PR?

Indeed, they are fixed in current master, you can rebase

@scottfgjordan
Copy link
Contributor Author

Replaced with: #11787

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants