slayercat / GoSNMPServer

GoSNMPServer is an SNMP server library fully written in Go. It provides Server Get, GetNext, GetBulk, Walk, BulkWalk, Set and Traps. It supports IPv4 and IPv6, using SNMPv2c or SNMPv3.
BSD 2-Clause "Simplified" License
90 stars 51 forks source link

bug:oidToByteString #17

Closed ZaneXo closed 1 year ago

ZaneXo commented 1 year ago
  1. When converting certain byte values ​​to the rune type, some byte values ​​may not be represented correctly because the rune type represents a Unicode character. This can cause byte values ​​to become unpredictable or incorrect after conversion.
  2. Each time a new request is received, such as Get, Get-Next, it will perform multiple oidToByteString operations, there is additional cpu overhead,but this is not necessary.
  3. If it is to verify the legitimacy of the OID, is it more appropriate to only perform the verification in SyncConfig, not when subsequent new requests flow in?
slayercat commented 1 year ago

It's so kind of you to help.