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
inverse matching of tags. Issue #10186 #4164
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs a couple minor adjustments. Applies and works OK otherwise.
@@ -521,6 +525,10 @@ function is_aoadv_used($rule_config) { | |||
$input_errors[] = gettext("Invert match cannot be selected with 'any'."); | |||
} | |||
|
|||
if ((isset($_POST['nottagged']) && empty($_POST['tagged']))) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is an extra set of parenthesis on the outside this test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
$group->add(new Form_Checkbox( | ||
'nottagged', | ||
'nottagged', | ||
'Invert', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be Invert match
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done,
but now GUI is not so pretty
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see that now, it forces a word wrap. I guess it's OK to move it back to just 'Invert' and adjust the text below to match.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
$pconfig['tagged'] | ||
))->setWidth(4); | ||
|
||
$group->setHelp('A packet can be matched on a mark placed before on another rule.'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change this to:
Match a mark placed on a packet by a different rule with the Tag option. Check Invert match to match packets which do not contain this tag.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
1744dc8
to
eb4e805
Compare
allow to inverse tagged rules
i.e.
block out quick on RED_NET ! tagged RED_NET_OK