Skip to content

Add an integration test for non-standard AES256 with Triple DES. - #80

Merged
mashhurs merged 2 commits into
logstash-plugins:mainfrom
mashhurs:aes256-with-3des-integration-test
Oct 9, 2025
Merged

mashhurs merged 2 commits into
logstash-plugins:mainfrom
mashhurs:aes256-with-3des-integration-test

Conversation

@mashhurs

@mashhurs mashhurs commented Oct 7, 2025

Copy link
Copy Markdown
Contributor

This will utilize snmpsim behind.

See this PR description for how the feature was tested with snmpsim - #78

@mashhurs mashhurs linked an issue Oct 7, 2025 that may be closed by this pull request
@mashhurs mashhurs self-assigned this Oct 7, 2025
@mashhurs
mashhurs requested a review from edmocosta October 7, 2025 22:17
Comment thread .ci/Dockerfile.snmpsim Outdated
# Start the SNMP simulator with the specified configuration
ENTRYPOINT ["snmpsim-command-responder", \
"--data-dir=./data", \
"--agent-udpv4-endpoint=0.0.0.0:163", \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should use the $PORT value here, so the argument works as expected.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, I was aware but forgot to make this change. fixed with 3487717

@mashhurs
mashhurs requested a review from edmocosta October 8, 2025 14:55
@mashhurs
mashhurs merged commit 8011e85 into logstash-plugins:main Oct 9, 2025
3 checks passed
@mashhurs
mashhurs deleted the aes256-with-3des-integration-test branch October 9, 2025 15:01
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.

Setup snmpsim to test AES256 with Triple-DES standard.

2 participants