-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Can this plugin be extended to receive snmp traps? #44
Comments
Hi edperry, Thanks |
I am not the repo manager, but it seems the reason the logstash-input-snmptrap did not implement is because the "gem" snmp library does not support it. I don't know if both are using the same library or not. This a bet would be that they are. As for adding it to this project, IMO it would not be appropriate, since this is a "Polling" plugin and receiving SNMP traps is a Passive operation (Listening) those are completely 2 different functions and again IMO should be 2 seperate functions like logstash-input-file and logstash-input-tcp similar functionality but different logic and behavior. but again, these are only my thoughts based on my observation of the Logstash community over the years. I would defer to any of the repo-admins to their opinions. |
@tchenscw Pretty much as @edperry says. The logstash-input-snmptrap is a purely ruby implementation using the http://github.com/hallidave/ruby-snmp gem for the SNMP parts whereas this plugin is JRuby using the SNMP4J Java library. The problem arises because (last I checked) the author of the ruby gem said back in 2011 that he didn't have the time or motivation to add it (hallidave/ruby-snmp#2 (comment)) in writing v3 support. As for including SNMP trap support in this plugin, whilst @colinsurprenant does have it listed in the 'Post GA' plans in the META issue logstash-plugins/logstash-input-snmptrap#2, I'd be inclined to agree with @edperry that this plugin is more designed as a poller not a listener. |
@tchenscw @edperry @axrayn thanks for your input here!
|
Ironically, after I posted, I started looking at whether I could rewrite the snmptrap plugin from scratch using the SNMP4J library. I stopped after the third or fourth "I'll just borrow this function from the snmp plugin...." and had realised the craziness. :) |
@axrayn, any PRs for that are welcome 😄 I myself likely won't be able to work on that in the next few weeks. |
@colinsurprenant I'm still struggling with tables, that's a higher priority for our needs at work. Will definitely see what I can do though. |
@axrayn finding the best way to represent snmp tables is definitely high on my priorities too. I'd love to hear your thoughts, let's open a separate "design" issue about this. |
@colinsurprenant @axrayn |
Issue #46 discusses trap handling. Closing. |
Hi,
Since snmpv3 support for snmptrap plugin has been opened for a long time, I was wondering whether there is a way to extend this plugin to receive snmp traps via v3 protocol.
The text was updated successfully, but these errors were encountered: