Edit report at http://bugs.php.net/bug.php?id=53594&edit=1
ID: 53594 Updated by: [email protected] 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: It is much easier and cleaner to simply go straight to OO then, without procedural API, and keep the old for legacy apps. Previous Comments: ------------------------------------------------------------------------ [2010-12-29 18:17:11] lytboris at gmail dot com 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. ------------------------------------------------------------------------ [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. ------------------------------------------------------------------------ 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
