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

pimd,pim6d: require router alert configuration #18202

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

Conversation

rzalamena
Copy link
Member

Implementation of the configuration require-router-alert for IGMP/MLD.

Until now there were two distinct behaviors between IGMP and MLD:

  1. IGMP always accepted reports even with no router-alert
  2. MLD always rejected reports without router-alert

This PR introduces a knob called "require-router-alert", which is disabled by default (meaning IGMP and MLD will accept all reports even without router alert) , that when it is enabled it will make PIM only accept IGMP/MLD reports with the router-alert option.

eqvinox and others added 4 commits February 25, 2025 13:07
Allows dropping any IGMP/MLD packets without the Router Alert IP option set.

Signed-off-by: David Lamparter <[email protected]>
Import scripts that send generated IGMP or MLD packets through
interfaces.

Signed-off-by: Rodrigo Nardi <[email protected]>
Add new tests for IGMP/MLD router alert checks.

Signed-off-by: Rafael Zalamena <[email protected]>
Let user know about the IGMP/MLD require router alert configuration.

Signed-off-by: Rafael Zalamena <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants