-
Notifications
You must be signed in to change notification settings - Fork 799
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
[backend] validate saved filters on create/edit (#4536) #4898
Conversation
…lerIdsToResolveForCache (#2686)
* with respect to isStixMatchFilterGroup capabilities
@@ -36,6 +37,11 @@ const checkFeedIntegrity = (input: FeedAddInput) => { | |||
} | |||
}); | |||
} | |||
|
|||
// our stix matching is currently limited, we need to validate the input filters | |||
if (input.filters && input.filters !== '{}') { |
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.
can input.filters be equal to {} ? I think it's undefined or of type FilterGroup (mode is compulsory etc)
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.
it can be apparently. I tested it, but don't remember the actual test case.
Well in fact this check is not useful, the validation would throw an error properly anyway..
03a86b8
to
c34f8ba
Compare
Reviewed lived and merged. |
We need to validate each and every filter saved into the platform with respect to isStixMatchFilterGroup capabilities:
Proposed changes
Related issues
Further comments
Only for internal review in the "filter team".