Project

General

Profile

Actions
Copy link

Optimization #3827

closed
VJ PA

output: clean up logging initialization code

Optimization #3827: output: clean up logging initialization code

Added by Victor Julien over 5 years ago. Updated 12 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Philippe Antoine
Target version:
8.0.0-beta1
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 (2 open — 11 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 IDsClosedPhilippe AntoineActions
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 - Feature #6090: eve/alert: missing dcerpc metadataClosedPhilippe AntoineActions
Related to Suricata - Feature #6091: eve/alert: missing dhcp metadataAssignedOISF DevActions
Related to Suricata - Bug #6092: eve/alert: missing pgsql metadataClosedJuliana Fajardini ReichowActions
Related to Suricata - Bug #6400: eve: log of DNS answer is in wrong direction ClosedJason IshActions
Blocks Suricata - Task #4683: detect: remove sigmatch_table in favor of a dynamic storage optionClosedPhilippe AntoineActions
Blocks Suricata - Feature #3958: enip: convert protocol parser to rustClosedPhilippe AntoineActions
Blocks Suricata - Feature #2695: websocket supportClosedPhilippe AntoineActions
Blocks Suricata - Feature #5773: doh: support DNS over HTTPS (DoH)ClosedPhilippe AntoineActions

VJ Updated by Victor Julien over 3 years ago Actions
Copy link
 #1

  • Target version changed from 7.0.0-beta1 to 7.0.0-rc1

VJ Updated by Victor Julien over 3 years ago Actions
Copy link
 #2

  • Target version changed from 7.0.0-rc1 to 8.0.0-beta1

PA Updated by Philippe Antoine almost 3 years ago Actions
Copy link
 #3

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

JI Updated by Jason Ish over 2 years ago Actions
Copy link
 #4

  • Assignee changed from Jason Ish to Philippe Antoine

PA Updated by Philippe Antoine over 2 years ago Actions
Copy link
 #5

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

PA Updated by Philippe Antoine over 2 years ago Actions
Copy link
 #6

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

PA Updated by Philippe Antoine over 2 years ago Actions
Copy link
 #7

  • 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 )

VJ Updated by Victor Julien over 2 years ago Actions
Copy link
 #8

  • Target version changed from 7.0.1 to 8.0.0-beta1

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

PA Updated by Philippe Antoine over 2 years ago Actions
Copy link
 #9

PA Updated by Philippe Antoine over 2 years ago Actions
Copy link
 #10

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

PA Updated by Philippe Antoine over 2 years ago Actions
Copy link
 #11

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

PA Updated by Philippe Antoine over 2 years ago Actions
Copy link
 #12

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

PA Updated by Philippe Antoine over 2 years ago Actions
Copy link
 #13

  • Related to Feature #6090: eve/alert: missing dcerpc metadata added

PA Updated by Philippe Antoine over 2 years ago Actions
Copy link
 #14

  • Related to Feature #6091: eve/alert: missing dhcp metadata added

PA Updated by Philippe Antoine over 2 years ago Actions
Copy link
 #15

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

PA Updated by Philippe Antoine over 2 years ago Actions
Copy link
 #16

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

PA Updated by Philippe Antoine about 2 years ago Actions
Copy link
 #17

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

PA Updated by Philippe Antoine about 2 years ago Actions
Copy link
 #18

  • Blocks Feature #3958: enip: convert protocol parser to rust added

PA Updated by Philippe Antoine about 2 years ago Actions
Copy link
 #19

PA Updated by Philippe Antoine about 2 years ago Actions
Copy link
 #20

PA Updated by Philippe Antoine almost 2 years ago Actions
Copy link
 #21

  • Status changed from In Review to Closed

VJ Updated by Victor Julien 12 months ago Actions
Copy link
 #22

  • Subject changed from clean up logging initialization code to output: clean up logging initialization code
Actions
Copy link

Also available in: PDF Atom