Skip to content

Update Suricata TA and Related Analytics#3974

Open
nasbench wants to merge 5 commits intodevelopfrom
update-suricata-ta
Open

Update Suricata TA and Related Analytics#3974
nasbench wants to merge 5 commits intodevelopfrom
update-suricata-ta

Conversation

@nasbench
Copy link
Contributor

@nasbench nasbench commented Mar 26, 2026

The old Suricata TA was nuked out from Splunkbase for "unknown reasons". We've identified a replacement TA that does the same parsing (overall). This PR aims to use that TA and update corresponding DS and contentctl.yml files. Below is the summary of the changes

Summary

Because the new TA has a different parsing for the URL field (and a bunch of other fields) we had to make some changes. The CCX TA uses an eval to get the value of the URL CIM field, as follow

EVAL-url = if(
    isnotnull('http.url')
    AND
    match('http.url',"^(?:https?\:\/\/)?([^\/\:]+).*$"),'http.url',
    
    if(isnotnull('http.url')
    AND
    isnotnull('http.hostname'),
    tostring('http.hostname')+tostring('http.url'),
    
    if(isnotnull('http.url')
    AND
    isnull('http.hostname'),
    'http.url',null())))

Which means the URL is usually a combination of http.url and http.hostname.

In contrast, the old TA was simply doing an alias to URL form http.url (which is technically incorrect). Since http.url actually contains the URI.

In practice, this means that we have to add a wildcard at the start of most of the analytic using the URL logic. Since it will be prefixed with the hostname. Most of the time this will not be an issue as long as the anchor string is long. I've only identified a couple of analytics suffering from small anchors and have proposed a fix. See details below.

Updated Analytics

As explained above most analytics have seen an addition of a wildcard character at the start of the string. I also took the time to update their metadata when needed and beautified the YAML/SPL. I will only detail meaningful fixes below.

  • Disable Schedule Task / Wscript Or Cscript Suspicious Child Process both of these analytics we demoted to Anomaly but never saw their score reduced. Hence I've updated them here.
  • Adobe ColdFusion Access Control Bypass - Update the logic to remove non double slash URLs. Since according to the reference they are not needed.
  • HTTP Request to Reserved Name on IIS Server - This analytic suffered from a short anchor logic. Hence I've updated the search to reduce FP by adding an extra check after the tstats.

Deprecated Analytics

  • Ivanti Sentry Authentication Bypass - After checking the reference, there is no strong IOC to catch this behavior and the actual analytic is weak. Hence it was deprecated.

Note: As a side bonus in theory all of these changes will work with the old TA if someone still has it installed.

@nasbench nasbench marked this pull request as ready for review March 26, 2026 19:49
@nasbench nasbench marked this pull request as draft March 27, 2026 11:01
@nasbench
Copy link
Contributor Author

Marking as draft, until i get the fixes deployed.

@nasbench nasbench added this to the v5.25.0 milestone Mar 27, 2026
@nasbench nasbench marked this pull request as ready for review March 27, 2026 14:50
@nasbench nasbench changed the title Update Suricata TA Update Suricata TA and Related Analytics Mar 27, 2026
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

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