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

DNP3: Convert logging to JsonBuilder - v1 #5167

Closed
wants to merge 6 commits into from

Conversation

jasonish
Copy link
Member

Converts DNP3 to use JsonBuilder for EVE logging.

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

suricata-verify-pr: 269

Some changes were made to the generated files instead of the
generator script. Update the script to generate what is
in the current state of the in-tree generated files.
First step of converting DNP3 to JsonBuilder by first converting
the non-generated code.
New methods for setting and appending float values.
Migration from Jansson to JsonBuilder.
@jasonish jasonish requested review from victorjulien and a team as code owners July 10, 2020 22:42
This was referenced Jul 11, 2020
@victorjulien
Copy link
Member

Merged in #5170, thanks!

@jasonish jasonish deleted the jsonbuilder/dnp3/v1 branch July 12, 2020 04:45
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 1, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 1, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 4, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 4, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 4, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 4, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 4, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 4, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 4, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 4, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 4, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 4, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 6, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 6, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Jul 6, 2022
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Jul 28, 2022
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Jul 28, 2022
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Jul 28, 2022
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Jul 28, 2022
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Jul 28, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Sep 23, 2022
benignbala pushed a commit to benignbala/suricata that referenced this pull request Nov 12, 2022
benignbala pushed a commit to benignbala/suricata that referenced this pull request Nov 12, 2022
benignbala pushed a commit to benignbala/suricata that referenced this pull request Nov 12, 2022
benignbala pushed a commit to benignbala/suricata that referenced this pull request Nov 12, 2022
benignbala pushed a commit to benignbala/suricata that referenced this pull request Nov 12, 2022
catenacyber added a commit to catenacyber/suricata that referenced this pull request Dec 2, 2022
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Dec 3, 2022
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.

2 participants