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

Fix ipfix buffer use and mirror test #197

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

themiron
Copy link

Without raw capabilities test does nothing, but with them fails later on buffer put with short crafted body or IPv4 send permission in IPv6-only environment

@swills
Copy link
Collaborator

swills commented Aug 17, 2024

The buffer fix test makes sense in ipfix_unix.go makes sense, I was able to reproduce that issue and confirm that change fixes it, but I did have to also set opts.IPFIXUDPSize inside the test.

I'm unsure about the changes to the test in ipfix_test.go. When I pull those in, I get a timeout on the test. I suspect this is an environment issue. I have IPv6 addresses setup and tried removing the v4 addresses on both loop-back and other interfaces in my test environment, but didn't have any luck. Can you explain more about your IPv6-only test environment that triggers this issue?

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.

2 participants