Skip to content

Create rule: Generic Financial Document Template#4498

Open
missingn0pe wants to merge 13 commits into
mainfrom
missingn0pe.fn.ESC-13584.new_rule_generic_financials_template
Open

Create rule: Generic Financial Document Template#4498
missingn0pe wants to merge 13 commits into
mainfrom
missingn0pe.fn.ESC-13584.new_rule_generic_financials_template

Conversation

@missingn0pe

Copy link
Copy Markdown
Member

Description

Detects messages with generic 'dear sir/madam' greetings that reference payment releases & timelines, contain links with suspicious hosting or open redirects, and exhibit unusual recipient patterns such as self-sending or missing recipients.

Associated samples

- Sample 1

Associated hunts

- Hunt 1 (Shared Samples)
- Hunt 2 (Multi-hunt)

Detects messages with generic 'dear sir/madam' greetings that reference payment releases & timelines, contain links with suspicious hosting or open redirects, and exhibit unusual recipient patterns such as self-sending or missing recipients.
@missingn0pe missingn0pe requested a review from a team May 15, 2026 22:31
@missingn0pe missingn0pe requested a review from a team as a code owner May 15, 2026 22:31
@github-actions github-actions Bot added the in-test-rules PR is in our testing suite to collect telemetry label May 15, 2026
github-actions Bot added a commit that referenced this pull request May 15, 2026
github-actions Bot added a commit that referenced this pull request May 15, 2026
…ncial document and suspicious hosting template
github-actions Bot added a commit to IndiaAce/sublime-rules that referenced this pull request May 19, 2026
…ancial document and suspicious hosting template
@missingn0pe

Copy link
Copy Markdown
Member Author

Telemetry looks good. Low volume TTP but viable. 4 net new samps over L90D, good detection in depth.

One observable - There is similar style rule that matches after a few changes, but only hits 2 of the 91 samps flagged in this PR over L90D, this PR does not lean on profiles, and is a very specific pattern.

- Hunt 1 L90D (Shared Samps)
- Hunt 2 L30D (Multi-hunt)

@missingn0pe missingn0pe added the review-needed Indicates that a PR is waiting for review label May 22, 2026
@missingn0pe missingn0pe self-assigned this May 27, 2026
@zoomequipd zoomequipd requested review from IndiaAce and removed request for a team May 27, 2026 15:05

@IndiaAce IndiaAce left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hey dude, the telemetry on this rule is crazy good in Mode! batting 1000 for malicious in the last 14 days. A few things I want to point out: I see in the latest batch of rules that almost all of them match existing rules, I'm super down for detection in-depth, especially when we are resolving FNs, but I wanted to see if you had considered modifying the existing Business Email Compromise (BEC) attempt from untrusted sender rule to account for this? Been looking through the hunts and I see that a good amount of these similar emails matching the BEC rule so just wanted to toss it out.

Also, I've been working on a suite of rules for that excessive padding in the body of the email and made a PR to modify that rule and the existing generic document sharing rule to catch this sample as well, so we've got good detection-in-depth coming for this. #4556

Comment thread detection-rules/link_financial_document_timeline_template.yml Outdated
Comment thread detection-rules/link_financial_document_timeline_template.yml Outdated
@IndiaAce

Copy link
Copy Markdown
Member

Gonna remove review-needed from this but feel free to shoot me a DM when you're ready for re-review for anything here!

@IndiaAce IndiaAce removed the review-needed Indicates that a PR is waiting for review label May 28, 2026
@missingn0pe

Copy link
Copy Markdown
Member Author

Hey dude, the telemetry on this rule is crazy good in Mode! batting 1000 for malicious in the last 14 days. A few things I want to point out: I see in the latest batch of rules that almost all of them match existing rules, I'm super down for detection in-depth, especially when we are resolving FNs, but I wanted to see if you had considered modifying the existing Business Email Compromise (BEC) attempt from untrusted sender rule to account for this? Been looking through the hunts and I see that a good amount of these similar emails matching the BEC rule so just wanted to toss it out.

Also, I've been working on a suite of rules for that excessive padding in the body of the email and made a PR to modify that rule and the existing generic document sharing rule to catch this sample as well, so we've got good detection-in-depth coming for this. #4556

Thanks! Given it's growing hit count, I found it a viable candidate for surfacing based on campaign template structure. I compared Business Email Compromise (BEC) attempt from untrusted sender rule against this sample, this sample does not fire nlu as BEC, it fires as cred_theft, which invalidated the rule as it is the primary condition of the rule.

Few updates:

- Removing starting check of strict greeting.
- Updating undisclosed recipients to current standard.
github-actions Bot added a commit that referenced this pull request Jun 3, 2026
github-actions Bot added a commit to IndiaAce/sublime-rules that referenced this pull request Jun 3, 2026
…financial document and suspicious hosting template
github-actions Bot added a commit that referenced this pull request Jun 3, 2026
…inancial document and suspicious hosting template
Noticed FP's from removing greeting condition. Adding nlu with character limiter to negate FP's and FN's.
Wrong logic committed. This is the correct logic
@missingn0pe

missingn0pe commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

github-actions Bot added a commit to IndiaAce/sublime-rules that referenced this pull request Jun 3, 2026
…financial document and suspicious hosting template
github-actions Bot added a commit that referenced this pull request Jun 3, 2026
github-actions Bot added a commit that referenced this pull request Jun 3, 2026
…inancial document and suspicious hosting template
A few changes after telemetry:

- The "will be released" branch now requires a specific date format (day of week + full date) within 15 chars, rather than matching on "will be released" alone.
- Tightened [1-4].[1-4] to [1-4]\W[1-4] to mitigate FP's.
- Changed display.text triggers
- Changed NLU confidence to mitigate scope creep
- Additional open redir trigger
- Removed goofy nlu filter that was mitigating a miss.

New free file host domain will be submitted for remaining misses.
github-actions Bot added a commit that referenced this pull request Jun 11, 2026
github-actions Bot added a commit that referenced this pull request Jun 11, 2026
…inancial document and suspicious hosting template
@missingn0pe

Copy link
Copy Markdown
Member Author

A few changes after telemetry:

  • The "will be released" branch now requires a specific date format (day of week + full date) within 15 chars, rather than matching on "will be released" alone.
  • Tightened [1-4].[1-4] to [1-4]\W[1-4] to mitigate FP's.
  • Changed display.text triggers
  • Changed NLU confidence to mitigate scope creep
  • Additional open redir trigger
  • Removed goofy nlu filter that was mitigating a miss.

New free file host domain will be submitted for remaining misses.

Hunts:

- Hunt 1 (Shared Samps) - L30D - PR logic - 202 hits, 7 net new hits
- Hunt 2 (Shared Samps) - L30D - New vs old logic
- Hunt 3 (Shared samps) - L30D - Old vs new logic
- Hunt 4 (Multi) - L30D - PR logic
- Hunt 5 (Multi) - L30D - New vs old logic
- Hunt 6 (Multi) - L30D -Old vs new logic

@missingn0pe

Copy link
Copy Markdown
Member Author

Can let bake for longer in test rules if needed as volume is still relatively low (only one new hit since yesterday in mode). Pushing to review needed for any additional changes needed!

@missingn0pe missingn0pe added the review-needed Indicates that a PR is waiting for review label Jun 12, 2026

@IndiaAce IndiaAce left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

HOLY telemetry my dude these results look great. A few things to consider, these should be relatively small changes but I'm still going to remove review-needed but feel free to chuck it back my way whenever you're ready for a new review. Here's a hunt showing the diff for my suggested changes: https://platform.sublime.security/messages/hunt?huntId=019ebde3-400c-75f0-bdf0-c92402c2d30b lmk your thoughts!

Comment thread detection-rules/link_financial_document_timeline_template.yml Outdated
Comment thread detection-rules/link_financial_document_timeline_template.yml
Comment thread detection-rules/link_financial_document_timeline_template.yml
@IndiaAce IndiaAce removed the review-needed Indicates that a PR is waiting for review label Jun 12, 2026
missingn0pe and others added 4 commits June 17, 2026 09:48
Removing suspicious hosting checks as it is limiting flagging on HTSRD hosting, shorteners & a recently added free file host domain.

Co-authored-by: Luke Wescott <69780712+IndiaAce@users.noreply.github.com>
Separating for readability.

Co-authored-by: Luke Wescott <69780712+IndiaAce@users.noreply.github.com>
Separating for readability.

Co-authored-by: Luke Wescott <69780712+IndiaAce@users.noreply.github.com>
Updating rule details to reflect changes made.
github-actions Bot added a commit that referenced this pull request Jun 17, 2026
Accidentally deleted " rule in name.
github-actions Bot added a commit that referenced this pull request Jun 17, 2026
…inancial document with proceedural greeting & timeline template
@missingn0pe missingn0pe requested a review from IndiaAce June 17, 2026 15:13
@missingn0pe missingn0pe added the review-needed Indicates that a PR is waiting for review label Jun 17, 2026
@missingn0pe missingn0pe enabled auto-merge June 17, 2026 15:14
@missingn0pe missingn0pe removed the review-needed Indicates that a PR is waiting for review label Jun 17, 2026
@missingn0pe missingn0pe added the review-needed Indicates that a PR is waiting for review label Jun 17, 2026
github-actions Bot added a commit that referenced this pull request Jun 17, 2026
github-actions Bot added a commit that referenced this pull request Jun 17, 2026
…inancial document with proceedural timeline template
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in-test-rules PR is in our testing suite to collect telemetry review-needed Indicates that a PR is waiting for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants