refactoring of the constant? about gosnmp HOT 6 CLOSED

Hey @tehnerd,

I think these were set by the original author as sensible defaults, they can of course be changed. If you, and the community at large feel these aren't actually sensible, then I suppose it makes sense to change them. Thoughts @soniah & @codedance?

from gosnmp.

I'll have to look at the code in detail...

• Nathan Owens [email protected] [2014-07-16 07:27:57 -0700]:

Hey @tehnerd,

I think these were set by the original author as sensible defaults, they can of course be changed. If you, and the community at large feel these aren't actually sensible, then I suppose it makes sense to change them. Thoughts @soniah & @codedance?

from gosnmp.

The defaults are there to support the Walk* methods.

I agree that MaxRepetitions should be configurable. I'll change this issue status to "enhancement" and submit a pull request for review.

Regarding OID: You can of course supply your own base OID (and the API encourages this). The default base is there to simply make it easier to not misuse! e.g.

mysnmp.Walk(".1")  // Walk from a very higher level

mysnmp.Walk("1.3.6.1.2.1") // Walk a specific table - e.g.  RFC 1213 system table

mysnmp.Walk("")  // Walk ???? - OK, let's default to the default public space as this should work on every standards compliant device.  Better than throwing a panic :-)

The design is analogous to net-snmp... from the man page:

If no OID argument is present, snmpwalk will search the subtree rooted at SNMPv2-SMI::mib-2

from gosnmp.

I've created a pull request of Sonia's master under #18 that should address requirements and close this issue. It's a low risk change but does hit public API surface so a review would be good.

Are we happy with the names?

As an aside:
My only other thinking would be to nest them at a deeper level such as

GoSNMP.Walk.MaxRepetitions
GoSNMP.Walk.NonRepeaters

but I feel the extra code complexity would offset any public surface area gain.

from gosnmp.

This was merged, I think we can close this unless @tehnerd has any other comments?

from gosnmp.

sounds good for me. thank you

from gosnmp.