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

Next/20190701/v3 #3985

Closed
wants to merge 14 commits into from
Closed

Conversation

Jason Ish and others added 14 commits June 28, 2019 09:00
If parsing the version fails, or no version is found, fail
instead of defaulting to a version of 0.0.0.
Remove it as it's (almost) never used. If it is really needed it can
be accessed through DetectEngineThreadCtx::tv as well.
Instead of the hardcode L4 matching in MPM that was recently introduced,
add an API similar to the AppLayer MPM and inspect engines.

Share part of the registration code with the AppLayer.

Implement for the tcp.hdr and udp.hdr keywords.
Remove unused field and macros.

Minor code style cleanups.
Inspects IPv6 header and extension headers.
@victorjulien victorjulien requested review from jasonish, norg, shivan1b and a team as code owners July 1, 2019 12:58
@victorjulien
Copy link
Member Author

Merged with a minor fixup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants