search

logstash-plugins / logstash-input-snmp

Apache License 2.0
17 stars 22 forks source link

Docs: example on setting IPv6 hosts #89

Closed kares closed 4 years ago

kares commented 4 years ago

including specs that verify IPv6 hosts are parsed fine by snmp4j

kares commented 4 years ago

seems to me like we do not need anything special to handle IPv6 hosts (https://github.com/logstash-plugins/logstash-input-snmp/pull/88) ... but maybe I've missed something.

colinsurprenant commented 4 years ago

Great! We should manually test it to make sure nothing is missing.

colinsurprenant commented 4 years ago

(until we have an integration tests setup in place - see #87).

kares commented 4 years ago

this is tested and confirmed to be working (the udp6: used by snmpd is not a valid syntax for snmp4j). happy to merge as is for docs to be picked up in the next release or do we want bump+changelog due the docs change?

colinsurprenant commented 4 years ago

➕1️⃣ bump+changelog!