netdisco / snmp-info

Other
35 stars 31 forks source link

CiscoStats.pm modification for Cisco PIX support #102

Closed netdisco-automation closed 7 years ago

netdisco-automation commented 16 years ago

without this patch Cisco PIX os/version shon as Unknown/Unknown. With this patch - PIXOS/7.2(2)

Patch below: diff -u -r1.17 CiscoStats.pm --- Info/CiscoStats.pm 5 Apr 2007 02:46:54 -0000 1.17 +++ Info/CiscoStats.pm 24 Aug 2007 15:05:55 -0000 @@ -89,6 +89,7 @@ # order here matters - there are Catalysts that run IOS and have catalyst in their description field. return 'ios' if ($descr =~ /IOS/); return 'catalyst' if ($descr =~ /catalyst/i); + return 'PIXOS' if ($descr =~ /PIX/i); return undef; }

@@ -103,9 +104,9 @@ }

# Newer Catalysts and IOS devices - if (defined $descr and $descr =~ m/Version (\d+\.\d+\([^)]+\)[^,\s]*)(,|\s)+/ ){ + if (defined $descr and $descr =~ m/Version (\d+\.\d+\([^)]+\)[^,\s]*)/ ){ return $1; - } + } return undef; }

Reported by: begemot

Original Ticket: snmp-info/patches/33

netdisco-automation commented 16 years ago

Logged In: YES user_id=97388 Originator: YES

PS. I'm not a perl/regex guru so please verify regex changes Description from PIX for example - "Cisco Cisco PIX Security Appliance Version 7.2(2)"

Original comment by: begemot

netdisco-automation commented 14 years ago

Original comment by: JeroenvIS

netdisco-automation commented 14 years ago

Functionality suggested by this patch seems to be integrated in CiscoStats.pm since v1.23; cleaning up this item.

Original comment by: JeroenvIS