[Bro-Dev] #912: Need option for ASCII logging writer to only print CSV header line
Bro Tracker
bro at tracker.bro-ids.org
Mon Nov 26 14:11:10 PST 2012
#912: Need option for ASCII logging writer to only print CSV header line
----------------------------+-------------------------------
Reporter: carsten | Owner:
Type: Merge Request | Status: new
Priority: Normal | Milestone: Bro2.2
Component: Bro | Version: git/master
Resolution: | Keywords: logging csv ascii
----------------------------+-------------------------------
Comment (by carsten):
Replying to [comment:14 seth]:
> > - I'm wondering if we should simply rename the option to "csv" instead
of
> > "only_single_header_row" ?
>
>
> That would imply comma separation though.
Personally I'm used to that "CSV" only means 0 or 1 header rows and
separated by "something", no guarantee for comma or semicolon or tab or
whatever. So I have no problem renaming it to CSV, as it does not imply
for me that a comma must be used. Originally I wanted to avoid this
discussion and named the option by what it actually does: printing only 1
header row and all other options like field separator have to be specified
separately. Bottom line: I don't mind either way.
> I haven't looked yet, but is there a way to make this a global setting?
(I know, I'm playing both sides of this argument)
Well, it *was* a global setting in my first patch, and then *someone* told
me it should not be a global but a per-writer option :-)
I rather think now that all the other options could be per-writer options
as well, rather than global.
--
Ticket URL: <http://tracker.bro-ids.org/bro/ticket/912#comment:16>
Bro Tracker <http://tracker.bro-ids.org/bro>
Bro Issue Tracker
More information about the bro-dev
mailing list