search

logstash-plugins / logstash-input-syslog

Apache License 2.0
37 stars 38 forks source link

Bump UDP recvfrom maxlen value to 65507 #38

Closed basoko closed 7 years ago

basoko commented 7 years ago

Increase the max length value passed to UDP's socket recvfrom method to 65,507 bytes, that is the max message length of the data in a UDP datagram.

basoko commented 7 years ago

I've noticed that the test that is not passing is due to a change in the LogStash::Util::Decorator.add_tags function, that now when this method is called initializes the tags with an empty array instead of nil. I could also update the test to make it pass.

jsvd commented 7 years ago

@basoko yes please, if you can include the fix to make specs pass I'll merge this asap

basoko commented 7 years ago

@jsvd Ok, it's fixed now.

jordansissel commented 7 years ago

LGTM

elasticsearch-bot commented 7 years ago

Jordan Sissel merged this into the following branches!

Branch Commits
master 340d23dbe47b2dac6ad11b89d7fc056da1090907