Skip to content

Fix TargetUserName computer account filter in computer service ticket detection#4147

Open
munzzyy wants to merge 1 commit into
splunk:developfrom
munzzyy:fix-computer-service-tickets-userprincipalname-filter
Open

Fix TargetUserName computer account filter in computer service ticket detection#4147
munzzyy wants to merge 1 commit into
splunk:developfrom
munzzyy:fix-computer-service-tickets-userprincipalname-filter

Conversation

@munzzyy

@munzzyy munzzyy commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Details

Fixes #4083. The Windows Large Number of Computer Service Tickets Requested detection excludes computer accounts with TargetUserName!="*$", but Event 4769 logs TargetUserName in userPrincipalName form (sAMAccountName@domain.tld) in some environments instead of the plain sAMAccountName. When that happens the trailing $ is gone and the filter never fires, so computer accounts still count toward the "large number of tickets" total and inflate the detection with false positives.

Fix strips everything after @ before checking for the $ suffix, same pattern already used in suspicious_kerberos_service_ticket_request.yml for the same field on the same event:

| eval TargetUserName = mvindex(split(TargetUserName, "@"), 0)
| search NOT TargetUserName="*$"

Bumped version to 12 and updated modification_date.

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.
  • Confirm updates to lookups are handled properly.

… detection

TargetUserName can come through as a userPrincipalName
(sAMAccountName@domain.tld) instead of a plain sAMAccountName, in which
case the existing TargetUserName!="*$" filter never matches and computer
accounts leak into the results. Normalize TargetUserName to the part
before the @ first, same approach already used in
suspicious_kerberos_service_ticket_request.yml, then filter on that.

Fixes splunk#4083
@patel-bhavin

Copy link
Copy Markdown
Contributor

@munzzyy - Thank you for proposing the fix. We are working on getting unit-testing to work on forks and will merge this PR after testing is done successfully.

@patel-bhavin patel-bhavin added this to the V6.3.0 milestone Jul 6, 2026
@munzzyy

munzzyy commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

Sounds good, thanks for the heads up. No rush on my end - let me know if there's anything you'd like me to change in the meantime.

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.

[BUG] UserPrincipalName issue in Windows Large Number of Computer Service Tickets Requested

2 participants