-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Applayer plugin 5053 v3.2 #11696
Applayer plugin 5053 v3.2 #11696
Conversation
instead of a global variable. For easier initialization with dynamic number of protocols
for expectation_proto Ticket: 5053
for alproto_names Ticket: 5053
Ticket: 5053
ERROR: ERROR: QA failed on ASAN_TLPR1_suri. Pipeline 22315 |
Ticket: 5053 delay after initialization so that StringToAppProto works
so that we can use safely EXCEPTION_POLICY_MAX*sizeof(x)
94b94a4
to
6547368
Compare
Draft to see CI with array dimensions well ordered.. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #11696 +/- ##
==========================================
- Coverage 82.63% 82.43% -0.21%
==========================================
Files 919 919
Lines 249032 248933 -99
==========================================
- Hits 205795 205201 -594
- Misses 43237 43732 +495
Flags with carried forward coverage won't be shown. Click here to find out more. |
ERROR: ERROR: QA failed on SURI_TLPR1_alerts_cmp. Pipeline 22316 |
Information: QA ran without warnings. Pipeline 22320 |
Clean in #11701 |
Link to ticket: https://redmine.openinfosecfoundation.org/issues/
Preliminary work for https://redmine.openinfosecfoundation.org/issues/5053
Describe changes:
#11572 next round
Still more work to do after :
Only
AppProtoStrings
is to be handled, but it is the big one.And then take remaining commits out of #11321
And supply an example of an app-layer plugin