Buffer for reading udp packets is too small. about statsd_exporter HOT 5 CLOSED

You're right, that should definitely be increased and the comment doesn't make too much sense. No idea why it was that low in the first place!

See #36

from statsd_exporter.

Thank you very much!

from statsd_exporter.

The docs default the max size to 512b0 for "open internet" safety concerns. But this can be configured by the admin.

This may be something we want to have as a runtime flag if possible.

from statsd_exporter.

Flag is overhead, I think.
Max UPD packet size is (65,535 − 8 byte UDP header − 20 byte IP header)
For message-based sockets, such as SOCK_DGRAM and SOCK_SEQPACKET, the entire message shall be read in a single operation.
So buffer[65,535] covers all possible cases. We allocate it just once, so fix is changing buffer to max allowable size. =)

from statsd_exporter.

Yea, that seems reasonable.

from statsd_exporter.