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

IP/Var + Suricata Fixes #544

Merged
merged 3 commits into from
Jun 13, 2024
Merged

IP/Var + Suricata Fixes #544

merged 3 commits into from
Jun 13, 2024

Conversation

coreyogburn
Copy link
Contributor

Suricata Overrides now show an IP/Var column in the tuning table.

The IP validation rules have been expanded to allow for $VARS and !$VARS in addition to IPv4 and IPv6 CIDRs. Hints have been updated to indicate support for vars. Links to where you set those variables have been added to the ip fields.

A bug where Suricata was dropping detection Ids on update has been fixed. Now a second update can happen without refreshing the page.

Updated tests.

Suricata Overrides now show an IP/Var column in the tuning table.

The IP validation rules have been expanded to allow for $VARS and !$VARS in addition to IPv4 and IPv6 CIDRs. Hints have been updated to indicate support for vars. Links to where you set those variables have been added to the ip fields.

A bug where Suricata was dropping detection Ids on update has been fixed. Now a second update can happen without refreshing the page.

Updated tests.
@coreyogburn coreyogburn force-pushed the cogburn/suricata-improvements branch from f441876 to 20321f0 Compare June 13, 2024 21:39
Added data-aid properties to the (now) gear icons that take you to suricata's vars in the config.

Updated column definitions to use i18n values.
@coreyogburn coreyogburn merged commit 71574f1 into 2.4/dev Jun 13, 2024
3 checks passed
@coreyogburn coreyogburn deleted the cogburn/suricata-improvements branch June 13, 2024 22:27
@github-actions github-actions bot locked and limited conversation to collaborators Jun 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants