Should default to \r\n on windows

[andrewvc]

When running on a windows box this should default to \r\n as a delimiter. This works pretty poorly ith the stdin input without this.

[pickypg]

I wonder if we can specify this so that it's the equivalent of \r?\n all of the time. I can't think of anyone that actually wants the \r in there, whether they're using a file or stdin.

[wiibaa]

One first improvment would be to use java System.lineSeparator() to set the default, feasible?

[jordansissel]

Maybe we make it a setting, maybe called "line_terminator" and have valid settings of "windows", "mac", "unix", or "network" ? (windows and network are identical, \r\n; mac is \r, unix is \n)

[pickypg]

But Mac doesn't do \r anymore. It does \n now. But, it is a good point.

As long as it's settable, then it really doesn't make a huge difference to me, but I think the default should really be "either of (\n|\r\n)" or "any of (\r|\n|\r\n)". There is an undocumented delimiter field that serves this purpose, but I think it requires it to be the exact delimiter rather than the match:

logstash-codec-line/lib/logstash/codecs/line.rb

Line 31 in ceee865

@buffer = FileWatch::BufferedTokenizer.new(@delimiter)

[andrewvc]

@jordansissel this plugin already has a 'delimiter' setting. I'm really curious if it works correctly with escaped characters in the logstash config however.

[andrewvc]

Also, the only reason delimiter is undocumented is that our doc generator hasn't run recently on this.

[andrewvc]

I think @wiibaa makes a great point, but I agree that we should look for either \r\n or \n. No one wants the \r.

[jordansissel]

Some Mac apps still use \r (FileMaker) even on OS X. :/

On Monday, June 20, 2016, Chris Earle notifications@github.com wrote:

But Mac doesn't do \r anymore. It does \n now. But, it is a good point.

As long as it's settable, then it really doesn't make a huge difference to
me, but I think the default should really be "either of (\n|\r\n)" or
"any of (\r|\n|\r\n)". There is an undocumented delimiter field that
serves this purpose, but I think it requires it to be the exact delimiter
rather than the match:

logstash-codec-line/lib/logstash/codecs/line.rb

Line 31 in ceee865

@buffer = FileWatch::BufferedTokenizer.new(@delimiter)

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#10 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAIC6o0ZGpvvhE9b57eM4DsrCCLWbzalks5qNurtgaJpZM4I6AdE
.