-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Filebeat] Improve ECS field mappings in panw module #17910
Conversation
Pinging @elastic/siem (Team:SIEM) |
- panw.panos.action - event.outcome, limit to succes/failure - event.kind - event.category, make array - event.type, make array - rule.name - related.user Closes elastic#16025
28f3d52
to
e3a48d4
Compare
@adriansr can you take a quick look at change in event.outcome (since you were author?) |
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.
@@ -62,6 +72,11 @@ | |||
"0.0.0.0", | |||
"0.0.0.0" | |||
], | |||
"related.user": [ | |||
"crusher", | |||
"crusher" |
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 wish the append
processor would deduplicate like the AppendTo
function in beats.
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
- panw.panos.action - event.outcome, limit to succes/failure - event.kind - event.category, make array - event.type, make array - rule.name - related.user Closes elastic#16025 (cherry picked from commit e174441)
What does this PR do?
Adds or changes the following fields in panw module:
Why is it important?
Improved ECS compliance makes data more useful in SIEM app and makes correlating between data sources easier.
Checklist
- [ ] I have commented my code, particularly in hard-to-understand areas- [ ] I have made corresponding change to the default configuration filesCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Related issues
Closes #16025