Consideration of initial Read() buffer size about gosnmp HOT 4 CLOSED

Distributions of OctetString size (bytes) across 20k+ values all from many different real-world devices (mostly printers and routers)

Mean: 10 bytes
Max: 256 bytes

from gosnmp.

I've done some more real world testing and experimenting. I'll put together a pull request to make a minor buffer size change building on @benjamin-thomas wonderful addition. In short, when we're walking (GetBulk) the pdu count is only 1, but the returned number of PDUs will be many more (MaxRepetitions). This leads to many retries. It's an easy fix.

Also while playing I found a Windows specific bug. A read on a buffer that's too small results in an error rather than a partial buffer read. I'll fix this in the same pull request.

from gosnmp.

OK. I've just filed a pull-request for @soniah to do a quick review (#46).

Summary of findings:

• A 1k initial buffer is fine for all real-world GET requests. The max size OctetString I found on a real-world device was 256 bytes.
• GETBULK needed some special handling.
• Interestingly SNMP4J and net-snmp have fixed buffer sizes like the old code.

from gosnmp.

Closed a while back in #46

from gosnmp.