Skip to content
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

SNMP Trap Collection (printing) #8520

Open
wants to merge 11 commits into
base: dev
Choose a base branch
from
Open

SNMP Trap Collection (printing) #8520

wants to merge 11 commits into from

Conversation

carloleo
Copy link

@carloleo carloleo commented Jul 15, 2024

Please sign (check) the below before submitting the Pull Request:

Link to the related issue.

Describe changes:

Added methods to SNMP.cpp to collect snmp traps. The implementation prints oid, mib, and value for all variables being in a SNMP trap.

carloleo and others added 9 commits July 4, 2024 21:42
	src/SNMP.cpp added new session to receive traps
	- added log
	- fixed typo
SNMP.cpp printed oid and mib in the new method
SNMP.cpp refactored code so that
	collectsTrap starts trap collection
	added dedicated select for traps
	removed trap session from vector
	- added loop to receive trap continuously
	- added comments and error logs
SNMP.h
	- added method to stop collecting trap
	- changed type of timeout parameter
	fixed memory deallocation
SNMP.h
	added space to force new version
@lucaderi
Copy link
Member

@carloleo This PR cannot be evaluated and accepted unless you sign ahead the CLA

@lucaderi
Copy link
Member

@carloleo Please fix the reported problems before we can evaluate your work

@carloleo
Copy link
Author

@lucaderi I just submitted the CLA. I'll sort issues out over the build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants