-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Add snmp_trap input plugin #6629
Conversation
Look up OIDs in SNMP traps using SnmpTranslate() from the SNMP plugin. Update dep for github.com/soniah/gosnmp to get Asn1BER stringer function. Add README.md.
to do:
|
Great job reimda. Some suggestions:
|
… interfaces to listen on
Thanks for the feedback @tesibelda
I added this as a tag called trap_mib to match the existing tag trap_name.
Since we already have the numeric IP address, I added it as a tag called source. Other inputs use the name "source" for this so it makes sense to follow convention. I agree a DNS name could be more useful than the IP but I'd like to wrap up this PR. Adding a setting to do reverse DNS sounds like a great follow-on feature. |
Required for all PRs:
closes #4377