-
Notifications
You must be signed in to change notification settings - Fork 474
windows: add file hash fields for the new Sysmon EID 29 #7015
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
Conversation
The current pipeline does not create the "file.hash" field for the new sysmon Event ID 29. As a result, that information is lost and does not end up on the event.
💚 CLA has been signed |
@fopson, please sign the CLA so that we can take a look at this. If you believe you have signed it, please check that the email address that you used agrees with the email address in the commits. |
Hi @efd6, I have signed the agreement twice now, but it is still showing up as unsigned. Not sure why. |
@fopson Looks good now. |
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.
@fopson Please add an entry to the changelog and update the manifest for the package. I expect that this change will require that the test expectations will need to be updated. Are you able to do this?
Done. |
/test |
🌐 Coverage report
|
Package windows - 1.28.0 containing this change is available at https://epr.elastic.co/search?package=windows |
The current pipeline does not create the "file.hash" field for the new sysmon Event ID 29. As a result, that information is lost and does not end up on the event.
What does this PR do?
It makes sure the file hash information from the new sysmon EID 29 is not lost when the "_temp" field is deleted.
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Sample Event
Screenshots