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

Add more validations #329

Merged
merged 2 commits into from Feb 26, 2018

Conversation

Projects
None yet
1 participant
@mstemm
Contributor

mstemm commented Feb 24, 2018

No description provided.

mstemm added some commits Feb 24, 2018

Add the ability to validate multiple rules files
Allow multiple -V arguments just as we do with multiple -r arguments.
With verbose output, print dangling macros/lists
Start tracking whether or not a given macro/list is actually used when
compiling the set of rules. Every macro/list has an attribute used,
which defaults to false and is set to true whenever it is referred to in
a macro/rule/list.

When run with -v, any macro/list that still has used=false results in a
warning message.

Also, it turns out the fix for
#197 wasn't being applied to
macros. Fix that.

@mstemm mstemm merged commit 38eb5b8 into dev Feb 26, 2018

3 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
sign-off-checker The commit doesn't require sysdig sign-off CLA because it belongs to mstemm part of draios/falco collaborators
Details

@mstemm mstemm deleted the add-more-validations branch Feb 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment