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

examples/sniffer: enable promiscuous, disable auto-ack #21244

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

benpicco
Copy link
Contributor

Contribution description

The sniffer should put network interfaces into promiscuous mode so they don't ifilter traffic.
It should also disable sending ACK messages for incomming frames.

Testing procedure

image

We now see all traffic between nodes.

Issues/PRs references

@github-actions github-actions bot added Area: drivers Area: Device drivers Area: examples Area: Example Applications labels Feb 25, 2025
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Feb 25, 2025
@benpicco benpicco requested review from miri64 and fabian18 February 25, 2025 16:09
@benpicco benpicco added the Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) label Feb 25, 2025
@riot-ci
Copy link

riot-ci commented Feb 25, 2025

Murdock results

✔️ PASSED

cf9be5a examples/sniffer: enable promiscuous, disable auto-ack

Success Failures Total Runtime
10271 0 10271 11m:59s

Artifacts

@mguetschow
Copy link
Contributor

Can you give an example of a packet that would currently be filtered out on master?

@benpicco
Copy link
Contributor Author

On master I only see broadcast traffic

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: drivers Area: Device drivers Area: examples Area: Example Applications CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants