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

Storm ingest snippets for open source threat feeds #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EXC3L-ONE
Copy link

This PR includes a few of my storm snippets for ingesting open source threat feeds.
Also thrown in my yara-family-namer which parses the malware family name from the YARA rule metadata.

Comment on lines +15 to +19
{[ou:org=$lib.gen.orgByFqdn("loobins.io")
:name="loobins"
:url="https://www.loobins.io/"
:desc="Living Off the Orchard: macOS Binaries (LOOBins) is designed to provide detailed information on various built-in macOS binaries and how they can be used by threat actors for malicious purposes."
]}
Copy link
Contributor

Choose a reason for hiding this comment

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

This subquery won't actually execute since there are no nodes in the pipeline here

Comment on lines +27 to +29
$body = $lib.json.load($body)

for $bin in $body {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
$body = $lib.json.load($body)
for $bin in $body {
for $bin in $resp.json() {

Comment on lines +40 to +42
for $path in $bin.paths {
[+(uses)> {[file:path=$path]}]
}
Copy link
Contributor

Choose a reason for hiding this comment

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

This loop will cause the it:prod:soft node to multiply, you can move the loop inside the edge add subquery (also more efficient that way)

Suggested change
for $path in $bin.paths {
[+(uses)> {[file:path=$path]}]
}
for $path in $bin.paths {
[+(uses)> {
for $path in $bin.paths {[file:path=$path]}
}]

}

// Create technique nodes
for $ttp in $bin.example_use_cases {
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure what the intended output of the macro is, but should probably move this loop into a subquery to avoid duplicate it:prod:soft nodes being yielded.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants