On Thu, 18 Feb 2016, Rainer Gerhards wrote:
Hi all,
I received a bug fix for impstats that changes the json-based formats
slighly. Please see:
https://github.com/rsyslog/rsyslog/pull/798
Any concerns?
I think we should go ahead and let JSON not have the @cee header. While it could
break someone's config, it's a small break that's easily fixed (either change
the type to cee instead of json, or fix their parser to accept json)
Long term, it's far cleaner than making cee and json be the same and then having
a _no_i_really_mean_json type as well.
If we are going to start down the road of mangling things at this level to
support elasticsearch, then we will end up adding lots of format variations
everywere. We really need a sanitize/cleanup that can fixup things to deal with
those sorts of problems. otherwise we're going to get creeping configs to handle
the issues all over the place.
David Lang
_______________________________________________
rsyslog mailing list
http://lists.adiscon.net/mailman/listinfo/rsyslog
http://www.rsyslog.com/professional-services/
What's up with rsyslog? Follow https://twitter.com/rgerhards
NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad of
sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you DON'T LIKE
THAT.
