Project

General

Profile

Actions

Optimization #3827

open

clean up logging initialization code

Added by Victor Julien over 3 years ago. Updated 8 months ago.

Status:
In Review
Priority:
Normal
Target version:
Effort:
Difficulty:
Label:

Description

Currently all loggers have setup and shutdown code that is very similar, and the rust loggers still have a fair bit of a C glue code as well.


Related issues 13 (10 open3 closed)

Related to Suricata - Bug #5977: eve/alert: missing KRB5 metadataClosedPhilippe AntoineActions
Related to Suricata - Task #4101: tracking: pluginsIn ProgressJason IshActions
Related to Suricata - Task #5053: app-layer: dynamic alproto IDsNewOISF DevActions
Related to Suricata - Bug #6501: eve/alert: missing TFTP metadataClosedPhilippe AntoineActions
Related to Suricata - Bug #6500: eve/alert: missing FTP metadataClosedPhilippe AntoineActions
Related to Suricata - Bug #6090: eve/alert: missing dcerpc metadataNewOISF DevActions
Related to Suricata - Bug #6091: eve/alert: missing dhcp metadataNewOISF DevActions
Related to Suricata - Bug #6092: eve/alert: missing pgsql metadataNewOISF DevActions
Related to Suricata - Bug #6400: log of DNS answer is in wrong direction NewEric LeblondActions
Blocks Suricata - Task #4683: detect: remove sigmatch_table in favor of a dynamic storage optionIn ReviewPhilippe AntoineActions
Blocks Suricata - Feature #3958: Convert protocol to Rust: ENIPIn ReviewPhilippe AntoineActions
Blocks Suricata - Feature #2695: websocket supportIn ReviewPhilippe AntoineActions
Blocks Suricata - Feature #5773: Support DNS over HTTPS (DoH)In ReviewPhilippe AntoineActions
Actions #1

Updated by Victor Julien over 1 year ago

  • Target version changed from 7.0.0-beta1 to 7.0.0-rc1
Actions #2

Updated by Victor Julien over 1 year ago

  • Target version changed from 7.0.0-rc1 to 8.0.0-beta1
Actions #3

Updated by Philippe Antoine 9 months ago

@Jason Ish would you assign me this ?
https://github.com/OISF/suricata/pull/8961 is doing this

Actions #4

Updated by Jason Ish 9 months ago

  • Assignee changed from Jason Ish to Philippe Antoine
Actions #5

Updated by Philippe Antoine 8 months ago

  • Status changed from Assigned to In Progress
  • Target version changed from 8.0.0-beta1 to 7.0.1
Actions #6

Updated by Philippe Antoine 8 months ago

  • Related to Bug #5977: eve/alert: missing KRB5 metadata added
Actions #7

Updated by Philippe Antoine 8 months ago

  • Status changed from In Progress to In Review

https://github.com/OISF/suricata/pull/9252 is a first PR, unblocking #5977
Second PR to close this would reusing these SimpleTxLogFunc from a JsonGenericLogger to remove many C files (as proposed in draft https://github.com/OISF/suricata/pull/8961 )

Actions #8

Updated by Victor Julien 8 months ago

  • Target version changed from 7.0.1 to 8.0.0-beta1

This type of cleanup work is not for a patch release.

Actions #9

Updated by Philippe Antoine 4 months ago

Actions #10

Updated by Philippe Antoine 4 months ago

  • Related to Task #5053: app-layer: dynamic alproto IDs added
Actions #11

Updated by Philippe Antoine 4 months ago

  • Related to Bug #6501: eve/alert: missing TFTP metadata added
Actions #12

Updated by Philippe Antoine 4 months ago

  • Related to Bug #6500: eve/alert: missing FTP metadata added
Actions #13

Updated by Philippe Antoine 4 months ago

  • Related to Bug #6090: eve/alert: missing dcerpc metadata added
Actions #14

Updated by Philippe Antoine 4 months ago

  • Related to Bug #6091: eve/alert: missing dhcp metadata added
Actions #15

Updated by Philippe Antoine 4 months ago

  • Related to Bug #6092: eve/alert: missing pgsql metadata added
Actions #16

Updated by Philippe Antoine 3 months ago

  • Related to Bug #6400: log of DNS answer is in wrong direction added
Actions #17

Updated by Philippe Antoine about 2 months ago

  • Blocks Task #4683: detect: remove sigmatch_table in favor of a dynamic storage option added
Actions #18

Updated by Philippe Antoine about 2 months ago

Actions #19

Updated by Philippe Antoine about 2 months ago

Actions #20

Updated by Philippe Antoine about 2 months ago

Actions

Also available in: Atom PDF