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

generic_e2e_test.go: Improve SnmpV3 tests #399

Closed
wants to merge 2 commits into from

Conversation

TimRots
Copy link
Member

@TimRots TimRots commented Mar 6, 2022

  • Fix broken TestSnmpV3AuthSHA512PrivAES192Get caused by nil passphrase in extendKeyBlumenthal cachekey generation.
  • Add TestSnmpV3AuthMD5PrivAES192Get test

Fixes: #395

Signed-off-by: Tim Rots tim.rots@protonmail.ch

Signed-off-by: Tim Rots <tim.rots@protonmail.ch>
Signed-off-by: Tim Rots <tim.rots@protonmail.ch>
@TimRots
Copy link
Member Author

TimRots commented Mar 6, 2022

--- FAIL: TestSnmpV3AuthMD5PrivAES192Get (30.03s)
    generic_e2e_test.go:420: Get() failed with error => request timeout (after 3 retries)

Build passes locally against Net-SNMP 5.9.
Will make another PR to update CircleCI to use 5.9, and get back to this

TimRots added a commit to TimRots/gosnmp that referenced this pull request Mar 8, 2022
* .circleci/config.yml: update apt sources.list to use impish packages
which will result in pulling in net-snmp 5.9 based snmpd.

Unblocks SNMPv3 testing of AES ciphers in gosnmp#399

Signed-off-by: Tim Rots <tim.rots@protonmail.ch>
@TimRots TimRots closed this Jan 18, 2024
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.

Only one of the two correct connection parameters can be correctly connected.
1 participant