Edit report at http://bugs.php.net/bug.php?id=53594&edit=1

 ID:                 53594
 Comment by:         lytboris at gmail dot com
 Reported by:        lytboris at gmail dot com
 Summary:            php-snmp rewrite
 Status:             Feedback
 Type:               Feature/Change Request
 Package:            SNMP related
 Operating System:   irrelevant
 PHP Version:        trunk
 Block user comment: N
 Private report:     N

 New Comment:

You have guessed my next target - OO interface. :-)



If you apply new patch you'll see there is very small piece of code
making SNMPv1 functions be polymorphic - dozen of lines, not more.

My thoughts are that old API should be deprecated sometime, so there
will be session-only support: both OO and non-OO.


Previous Comments:
------------------------------------------------------------------------
[2010-12-29 18:02:23] [email protected]

What do you think about keeping the old APIs as it is, froze it and add
a much nicer and flexible OO one instead? For any new improvements?
That's what I did for zip and brings much more rooms for new stuff while
reducing the maintenance work load.

------------------------------------------------------------------------
[2010-12-29 17:56:28] lytboris at gmail dot com

Reuploaded patches (to be allied against trunk)



+ session-like workflow:

   $session = snmp_session_open(...);

   $result = snmpget($oid);

   ...

   snmp_session_close($session);

+ support for SNMPv3 contextName & contextEngineID properties in new
syntax style



+ tests for new features



snmp(real)?{get,walk,set} functions are now polymorphic - they accepts
both old (SNMPv1) arguments and new session-like. All version protocols
are handled by the since protocol version is now set using
snmp_session_open.

------------------------------------------------------------------------
[2010-12-23 09:03:38] lytboris at gmail dot com

This patch covers bugs

#44193

#45893

#51336

------------------------------------------------------------------------
[2010-12-23 07:20:18] lytboris at gmail dot com

I know about version lifecycle. But Cacti 087 (Larry mentioned it) runs
on 5.2 branch.



Here you are patch for trunk. There is no difference between 5.3 and
trunk for now so this patch may be appied to 5.3 branch too.

------------------------------------------------------------------------
[2010-12-22 22:07:14] [email protected]

Thanks for this great patch :)



However 5.3 is in maintenance mode (and 5.2 is dead btw), please provide
a patch against trunk instead.

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    http://bugs.php.net/bug.php?id=53594


-- 
Edit this bug report at http://bugs.php.net/bug.php?id=53594&edit=1

Reply via email to