Project

General

Profile

Actions

Feature #1970

closed

json: make libjansson mandatory

Added by Victor Julien almost 8 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Effort:
Difficulty:
Label:

Description

The idea is to reduce the number of conditionals and json/eve is a core component of suricata these days.


Related issues 1 (0 open1 closed)

Related to Suricata - Bug #2473: Please allow disabling every option (add --without-{option})ClosedActions
Actions #1

Updated by Victor Julien over 7 years ago

  • Subject changed from make libjansson mandatory to json: make libjansson mandatory

Since Jason has some code to do JSON output w/o libjansson, this should be reconsidered.

Actions #2

Updated by Victor Julien over 7 years ago

  • Target version changed from 4.0beta1 to 70
Actions #3

Updated by Victor Julien over 7 years ago

  • Target version changed from 70 to TBD
Actions #4

Updated by Victor Julien over 6 years ago

  • Related to Bug #2473: Please allow disabling every option (add --without-{option}) added
Actions #5

Updated by Victor Julien over 6 years ago

  • Status changed from New to Assigned
  • Assignee changed from OISF Dev to Jason Ish

Jason, any thoughts on this? I think now with Rust support we've tied in more to jansson.

Actions #6

Updated by Jason Ish over 6 years ago

I'd be OK with making libjansson mandatory. There would have to be a good argument made to make it required I think.

Actions #7

Updated by Victor Julien over 6 years ago

I guess the only reason not to do it is if your own json output code would be used instead?

Actions #8

Updated by Jason Ish over 6 years ago

Victor Julien wrote:

Jason, any thoughts on this? I think now with Rust support we've tied in more to jansson.

With Rust mandatory, we could potentially move away from libjansson and do the eve output completely in Rust, using its own Rust support. Might be a bit of headache for logging app-layer protos that are done in C, but yeah, requiring Rust right now pretty much requires libjansson, if you want any logging.

Actions #9

Updated by Victor Julien about 6 years ago

  • Target version changed from TBD to 5.0beta1

When Rust is mandatory we need to revisit this. Either make jansson mandatory or replace it.

Actions #10

Updated by Victor Julien over 5 years ago

  • Status changed from Assigned to Closed
Actions

Also available in: Atom PDF