# [PATCH] SNMP value retrieval

- Thread: https://externals.io/message/870
- Original archive: https://news-web.php.net/php.internals/870
- Started by Harrie Hazewinkel on 2003-04-11T19:23:19+00:00
- Messages: 1
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #870 — Harrie Hazewinkel — 2003-04-11T19:23:19+00:00

URL: https://externals.io/message/870#870

HI,

Jonny, it your personal email address is bouncing.
That you know (maybe even the mailinglist email bounces for you).

I have looked at your patch and tested it. Seem OK.
some questions and changes I would like you feedback on below.




>
> Hi,
>
> as promised I've created a patch which adds some functionality to the
> SNMP module. It's controlled with a new function named
> snmp_set_valuemethod(int method) which changes the way how SNMP values
> are returned by snmpget(), snmpwalk() and snmpwalkoid(). The following
> (new) constants can be passed to snmp_set_valuemethod():
> - SNMP_VALUEMETHOD_LIBRARY:
> - SNMP_VALUEMETHOD_PLAIN:
> - SNMP_VALUEMETHOD_OBJECT:

Do these names to be so long?? I was thinking we cane drop the
'VALUEMETHOD' or 'METHOD' part. Would that be OK and which do
you prefer??


>  the value type returned by the SNMP
>   agent (the ASN_* constants from the net-snmp library are also
>   defined as PHP constants)

I believe the ASN_ prefix should be SNMP_ and changed as such.
That they are internally ASN_ does not matter.

ANother thing I would like to ask is whether the group wants to support
also all kinds of types of SNMP that are not standardized and somehow
added to NET-SNMP as an expiriment.
I believe it is better to drop those, what do you think??


Harrie



Harrie