Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change Logstash UDP default size limit #2111

Closed
domenkozar opened this issue Nov 20, 2014 · 8 comments
Closed

Change Logstash UDP default size limit #2111

domenkozar opened this issue Nov 20, 2014 · 8 comments

Comments

@domenkozar
Copy link

UDP RFC relevant text:

A field that specifies the length in bytes of the UDP header and UDP data. The minimum length is 8 bytes since that's the length of the header. The field size sets a theoretical limit of 65,535 bytes (8 byte header + 65,527 bytes of data) for a UDP datagram. The practical limit for the data length which is imposed by the underlying IPv4 protocol is 65,507 bytes (65,535 − 8 byte UDP header − 20 byte IP header).[2]

I propose to raise the default limit of UDP buffer to 65,507 bytes.

@jordansissel
Copy link
Contributor

@iElectric has this negatively impacted you?

@domenkozar
Copy link
Author

Yes, I spend a few hours on chasing a bug. I've fixed the bug on the client side to split messages, but then logstash was parsing JSON in a weird way if it hit the 8k limit.

@jordansissel
Copy link
Contributor

Did changing the read size in logstash fix your issue?

@domenkozar
Copy link
Author

Yes. But I see no reason why not to default to RFC value to prevent such issues.

@domenkozar
Copy link
Author

@jordansissel
Copy link
Contributor

@iElectric sorry, I'm not trying to be combative - just making sure that the proposed fix would resolve your problems. Thanks for clarifying :)

@domenkozar
Copy link
Author

🍻

@jordansissel
Copy link
Contributor

For Logstash 1.5.0, we've moved all plugins to individual repositories, so I have moved this issue to logstash-plugins/logstash-input-udp#1. Let's continue the discussion there! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants