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

Implement filtering in macfilter plugin #202

Closed
damyan opened this issue Oct 24, 2024 · 0 comments · Fixed by #265
Closed

Implement filtering in macfilter plugin #202

damyan opened this issue Oct 24, 2024 · 0 comments · Fixed by #265
Assignees
Labels
enhancement New feature or request

Comments

@damyan
Copy link
Member

damyan commented Oct 24, 2024

I want to be able to filter on existing list of MACs. Configuration proposal:

allowList: 
  - AA:BB:CC
  - FF:EE:00
denyList:
  - EE:00
  - AA:BB

The plugin is supposed to be configured first in the plugin chain. Either an allow, or a deny list (or both) should be specified.
A client MAC address shall be validated against both lists according to the following rules:

Allow List MAC Status Deny List MAC Status Client Action
match no match allow
match match deny
match not defined allow
no match no match deny
no match match deny
no match not defined deny
not defined no match allow
not defined match deny
not defined not defined not allowed, configuration error on startup
@damyan damyan added the enhancement New feature or request label Oct 24, 2024
@damyan damyan changed the title Support exclude list in metal plugin Support filter list in metal plugin Feb 7, 2025
@damyan damyan changed the title Support filter list in metal plugin Implement filtering in whitelist plugin Feb 7, 2025
@damyan damyan changed the title Implement filtering in whitelist plugin Implement filtering in macfilter plugin Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants