Skip to content

Detection fixes - #2260

Merged
josehelps merged 5 commits into
developfrom
DetectionFixes
Jun 22, 2022
Merged

josehelps merged 5 commits into
developfrom
DetectionFixes

Conversation

@pyth0n1c

Copy link
Copy Markdown
Collaborator

Fixes some detections that were failing in CI/CD because they didn't include custom_index AND fixes some detections that were failing (I think) because they included a stats field (user) that does not exist in the raw data. Also fixes a field which was renamed in a sysmon update.

Double-check that the meaning of the logic in the detection did not change.

pyth0n1c added 5 commits June 21, 2022 15:45
line.  It does not appear to exist in the raw
data and is not pulled into any field(s).
However, there is a field called userName
which, presumably, represents the same data.
that was created when custom_index
support did not yet exist.
since it does not exist and the field userName
does exist.  And it is likely to have captured
the same data.
file that was failing because it did not
have one.
@auto-assign
auto-assign Bot requested a review from MHaggis June 22, 2022 16:13
@josehelps
josehelps enabled auto-merge June 22, 2022 17:17
@josehelps
josehelps disabled auto-merge June 22, 2022 17:19
@josehelps
josehelps merged commit cda681e into develop Jun 22, 2022
@delete-merged-branch
delete-merged-branch Bot deleted the DetectionFixes branch June 22, 2022 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants