feat: Cisco Secure Firewall SnortML GID411 and dual-signal corroboration samples - #1206
feat: Cisco Secure Firewall SnortML GID411 and dual-signal corroboration samples#1206AAH20 wants to merge 2 commits into
Conversation
…ion samples Add unit-test datasets for ML-only (GeneratorID=411) intrusion events and classic signature + high EVE pairs sharing ConnectionID. Co-authored-by: Cursor <cursoragent@cursor.com>
|
CLA: signed (same contributor as security_content #4221 / issue #4220). |
nasbench
left a comment
There was a problem hiding this comment.
The log files have to be uploaded as lfs, and a corresponding yaml file needs to be provided. Please check out other entries in this repo for reference.
Here is the template for the YAML - https://github.com/splunk/attack_data/blob/master/datasets/TEMPLATE.yml
Also to me these seems like fake log. Were these generated by evidence forge?
Since I am seeing an EVE but no fingerprint for example, the URL field is missing for connection events, also the initiator port is reversed seeing that this is HTTP traffic.
This looks like slop to me :)
Addresses @nasbench's review on splunk#1206 -- every specific technical point was correct, verified against this repo's own real connection_event/connection_events.log (same author) as ground truth: - InitiatorPort/ResponderPort were reversed on every entry. For client-to-server HTTPS, the initiator (client) uses a high ephemeral port and the responder (server) uses 443 -- these samples had it backwards, which is exactly the kind of tell that reveals unvalidated synthetic data. - The ConnectionEvent was missing EVE_Fingerprint (the TLS ClientHello fingerprint Encrypted Visibility Engine detection is actually keyed on) and URL, both present on every real EVE-tagged connection event in this repo. - Also added the SSL_* fields (CertFingerprint, CipherSuite, FlowStatus, Version, ServerCertStatus, ActualAction, ExpectedAction) and Ingress/EgressInterface+Zone that accompany EVE telemetry in the real reference file, rather than a partial fix. - Replaced README.md with a proper dataset .yml (author/id/date/ description/mitre_technique/datasets), matching the convention every other cisco_secure_firewall_threat_defense sibling dataset uses. - Re-added the .log files through Git LFS per .gitattributes (datasets/**/*.log is LFS-tracked; they were previously committed as plain blobs). Signed-off-by: aah20 <aah20@users.noreply.github.com>
|
@nasbench Fair callout, and correct on every point — I checked each one against this repo's own real reference data (
On "slop" — you're right that it was. Reversed ports on synthetic data is exactly the kind of thing that only survives if nobody checked it against a real capture or a real reference in the same repo, and I hadn't. I have now, and the specific numbers above are why I'm not going to push back on the assessment. Ready for another look. |
Details
Adds unit-test datasets for dual-signal triage:
Supports ESCU detections proposed in splunk/security_content#4220
Notes
Made with Cursor