-
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
x-pack/winlogbeat: add parent process ID to new process creation events #31102
Conversation
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
@Mergifyio update |
✅ Branch has been successfully updated |
Is this a sync of the pipeline from elastic/integrations#2966? Given this is without tests for the moment I wanted to make sure this is the same such that we know it was tested. |
Yes, this is the sync. I'll also do the change mentioned here for this, but it's not without tests. There is a 4688 evtx that is ingested and tested. |
What's running that test (if you didn't update that golden file what would fail)? I thought these were without tests due to #30406. |
No, you are right. I have copied over the expectation from the integrations package, so when there are test this will be checked. Apologies. |
…ts (elastic#31102) Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
…ts (elastic#31102) Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
…ts (#31102) Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
What does this PR do?
Add parent process ID to new process creation events.
Why is it important?
Parent process ID is important information for security/threat detection.
Checklist
- [ ] I have made corresponding changes to the documentation- [ ] I have made corresponding change to the default configuration filesCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs