Actions
Feature #1970
closedjson: make libjansson mandatory
Description
The idea is to reduce the number of conditionals and json/eve is a core component of suricata these days.
Actions
Added by Victor Julien almost 9 years ago. Updated over 6 years ago.
Description
The idea is to reduce the number of conditionals and json/eve is a core component of suricata these days.
Since Jason has some code to do JSON output w/o libjansson, this should be reconsidered.
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.
When Rust is mandatory we need to revisit this. Either make jansson mandatory or replace it.