Feature #1970
closed
json: make libjansson mandatory
Added by Victor Julien over 7 years ago.
Updated about 5 years ago.
Description
The idea is to reduce the number of conditionals and json/eve is a core component of suricata these days.
- 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.
- Target version changed from 4.0beta1 to 70
- Target version changed from 70 to TBD
- Related to Bug #2473: Please allow disabling every option (add --without-{option}) added
- 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.
I'd be OK with making libjansson mandatory. There would have to be a good argument made to make it required I think.
I guess the only reason not to do it is if your own json output code would be used instead?
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.
- Target version changed from TBD to 5.0beta1
When Rust is mandatory we need to revisit this. Either make jansson mandatory or replace it.
- Status changed from Assigned to Closed
Also available in: Atom
PDF