-
Notifications
You must be signed in to change notification settings - Fork 407
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
[fortinet] Make event.original optional in fortinet #1075
Conversation
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
5dd8a19
to
707d74d
Compare
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪 |
707d74d
to
4ca5622
Compare
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.
LGTM, just a quick comment you should check first.
{{#if internal_interfaces.length}} | ||
{{else}} | ||
{{#if external_interfaces.length}} | ||
processors: |
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.
Is processors meant to be twice here?
{{#if internal_interfaces.length}} | ||
{{else}} | ||
{{#if external_interfaces.length}} | ||
processors: |
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.
Is processors meant to be twice here?
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 is added only once, in case any of internal_interfaces
and external_interfaces
are set, otherwise processors
can be empty and it was breaking the system tests. This if else if
is needed since handlebars lacks an ||
operator.
What does this PR do?
Makes event.original optional
Checklist
changelog.yml
file.- [] If I'm introducing a new feature, I have modified the Kibana version constraint in my package'smanifest.yml
file to point to the latest Elastic stack release (e.g.^7.13.0
).Related issues
Screenshots