Skip to content

Update get_domainuser_with_powershell_script_block.yml - #2345

Closed
TheLawsOfChaos wants to merge 1 commit into
splunk:developfrom
TheLawsOfChaos:patch-3
Closed

TheLawsOfChaos wants to merge 1 commit into
splunk:developfrom
TheLawsOfChaos:patch-3

Conversation

@TheLawsOfChaos

@TheLawsOfChaos TheLawsOfChaos commented Aug 29, 2022 •

Copy link
Copy Markdown
Contributor

Making fields consistent, replaced user_id with UserID to match extracted field type, and make it match other detections. A cosmetic change.

Details

What does this PR have in it? Screenshots are worth 1000 words 😄

Checklist

  • Validate name matches <platform>_<mitre att&ck technique>_<short description> nomenclature
  • CI/CD jobs passed ✔️
  • Validated SPL logic.
  • Validated tags, description, and how to implement.
  • Verified references match analytic.

Making fields consistent, replaced user_id with UserID to match extracted field type, and make it match other detections. A cosmetic change.
@auto-assign
auto-assign Bot requested a review from MHaggis August 29, 2022 16:46
@patel-bhavin

Copy link
Copy Markdown
Contributor

we will move this into a new branch to test the search

@patel-bhavin patel-bhavin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we get a the log file in XML format that we can test this detection against?

@patel-bhavin patel-bhavin added the WIP DO NOT MERGE Work in Progress label Sep 12, 2022
@patel-bhavin

Copy link
Copy Markdown
Contributor

this search update has been added to: #2366. Thank you!

@TheLawsOfChaos
TheLawsOfChaos deleted the patch-3 branch February 2, 2023 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

WIP DO NOT MERGE Work in Progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants