Skip to content
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

Update misnamed eventid #3144

Merged
merged 3 commits into from
Sep 27, 2024
Merged

Update misnamed eventid #3144

merged 3 commits into from
Sep 27, 2024

Conversation

pyth0n1c
Copy link
Collaborator

A number of YMLs had issues around the data_source object(s) in the YMLs.
Either the wrong named were used for Sysmon EventID (a space was accidentally included), the key in the YML was data_sources instead of data_source, or other minor issues.
This PR should resolve those issues.

of datasources. this was mostly removing spaces
to convert "Sysmon Event ID N" to
"Sysmon EventID N". Also fix some ymls
that used the field "data_sources" when
it should be "data_source"
Copy link
Contributor

@ljstella ljstella left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM-

Failing CI jobs are currently unavoidable.

Copy link
Contributor

@ljstella ljstella left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the change in detections/application/windows_ad_privileged_group_modification.yml require a version bump?

@pyth0n1c
Copy link
Collaborator Author

pyth0n1c commented Sep 27, 2024

Does the change in detections/application/windows_ad_privileged_group_modification.yml require a version bump?

Yes, nice catch! In fact this IS caught by the version-checking, it's just obfuscated by the other error on missing non-public content.
https://github.com/splunk/security_content/actions/runs/11060840382/job/30732226304#step:5:246

image

@ljstella ljstella self-requested a review September 27, 2024 21:08
Copy link
Contributor

@ljstella ljstella left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@pyth0n1c pyth0n1c merged commit 7c38b95 into develop Sep 27, 2024
4 of 8 checks passed
@pyth0n1c pyth0n1c deleted the update_misnamed_eventid branch September 27, 2024 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants