[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
 > > "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