-
Notifications
You must be signed in to change notification settings - Fork 1.6k
devguide: add chapter about exception policies - v1 #14483
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: main
Are you sure you want to change the base?
Conversation
Adds a chapter indicating what are the main steps when adding exception policies, how is it possible to extend them, as well as main aspects and files to consider when doing so. Task OISF#5612
| For an example PR on adding a new exception policy, see: | ||
| https://github.com/OISF/suricata/pull/7791, especially commit | ||
| `aa5bb2c329aff5 <https://github.com/OISF/suricata/pull/7791/commits/aa5bb2c329aff59b7811b43258ffd4d95fe7364f>`_. | ||
|
|
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.
Could possibly also add a section or note about the command-line options for simulating exception scenarios...
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #14483 +/- ##
==========================================
- Coverage 82.09% 82.08% -0.01%
==========================================
Files 1013 1013
Lines 262402 262402
==========================================
- Hits 215414 215398 -16
- Misses 46988 47004 +16
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
WARNING:
Pipeline = 28748 |
|
|
||
| Exception Policies allow control on how the engine will behave when it reaches | ||
| exception scenarios where it could lose visibility into traffic being processed, | ||
| for instance. |
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.
Feels strange to end a sentence by , for instance
|
|
||
| - hitting memory capacity limits; | ||
| - encountering application layer protocol errors; | ||
| - picking a TCP session midstream. |
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.
Can these exception policies be reachable from the rules language ? (from signatures)
Task #5612
Link to ticket: https://redmine.openinfosecfoundation.org/issues/
https://redmine.openinfosecfoundation.org/issues/5612
Built version of chapter: https://suri-rtd-test.readthedocs.io/en/devguide-5612-v1/devguide/extending/exception-policies/index.html
Describe changes: