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/184/20231117/v1 #9825

Closed
wants to merge 20 commits into from
Closed

Conversation

catenacyber and others added 20 commits November 17, 2023 15:50
This commit changes the conditional logging functions to use bool rather
than int values.
Change Suricata operational values from int to bool.
On a push of the same branch, cancel the previous running builds.yml
workflow.
The new author details will still be available in the artifact, we're
just not calling them out in a nighly visible pull request comment.
Flow_id explanation expanded from version shared by Peter Manev.

Task OISF#6445
Our tls fields not_after and not_before are actually logged as
`notafter` and `notbefore`, but were documented with the underscore.

Update the documentation, since updating the log format itself would be
a breaking change.

Task OISF#5494
This is an example of what adding plugin examples to the Suricata repo
could look like.

This plugin is an example plugin for an EVE filetype. It could be
extended to support outputs like Redis, syslog, etc.

There is one issue with adding plugins like this to an autotools
project, the project can't be built with --disable-shared, which is
more of an autotools limitation, and not really a Suricata issue.
Suricata built with --disable-shared will load plugins just fine.

Note that the examples directory was added as DIST_SUBDIRS as we don't
want normal builds to recurse into it and attempt to build the plugin,
its just an example, but we still need to keep distcheck happy.
@victorjulien
Copy link
Member Author

Didn't mean to include #9498

@victorjulien victorjulien deleted the next/184/20231117/v1 branch June 23, 2024 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

7 participants