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

This commit closes the issue #2033. #2189

Open
wants to merge 1 commit into
base: v2/master
from

Conversation

@studersi
Copy link

studersi commented Oct 23, 2019

Added a fix for issue #2033 based on #1353 and 9ac9ff8.

The fix is basically the same as is used for #1353, adding a sanity check after the arguments are parsed. I added the check for ctl:ruleRemoveTargetById as well as ctl:ruleRemoveTargetByMsg.

I'm currently unable to check this in the environment where it first occurred and I currently cannot reproduce the error as it was reported to me.

Note: This change also fixes inconsistent whites pace on the original fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.