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

Update CRYPTO_MINING_DOMAINS IOCs; add two additional tests #973

Merged
merged 5 commits into from
Nov 28, 2023

Conversation

egibs
Copy link
Contributor

@egibs egibs commented Nov 27, 2023

Background

This PR expands the set of CRYPTO_MINING_DOMAINS with many additional domains. To ensure correct support for subdomain-based IOCs, I also added two additional tests.

Changes

  • Adds additional cryptomining domains to the CRYPTO_MINING_DOMAINS set
  • Adds two new tests to ensure subdomain-based IOCs correctly alert

Testing

  • make fmt; make lint; make test

@egibs egibs requested review from a team November 27, 2023 15:07
CRYPTO_MINING_DOMAINS = {
"monerohash.com",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I also sorted this list alphabetically.

@@ -90,77 +90,262 @@
"${::-j", # example: ${${::-j}${::-n}di:${::-l}d${::-a}p://example.com:1234/callback}
}

# Sources:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't know where the original entries were sourced from, but I added the references to where I was sourcing the new entries from.

Copy link
Contributor

@arielkr256 arielkr256 left a comment

Choose a reason for hiding this comment

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

Nice update!

@egibs egibs enabled auto-merge (squash) November 28, 2023 17:10
@egibs egibs merged commit 02a16bb into main Nov 28, 2023
3 checks passed
@egibs egibs deleted the egibs-update-cryptomining-domain-iocs branch November 28, 2023 17:12
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