Skip to content

Update attachment_legal_pdf.yml - #5303

Open
missingn0pe wants to merge 1 commit into
mainfrom
missingn0pe.fn.ESC-24587.attachment_legal_pdf
Open

Update attachment_legal_pdf.yml#5303
missingn0pe wants to merge 1 commit into
mainfrom
missingn0pe.fn.ESC-24587.attachment_legal_pdf

Conversation

@missingn0pe

Copy link
Copy Markdown
Member

Description

Changes:

  • Adding "non-alphabet" condition to .producer FE extraction.
  • Adding "security check" condition to .final_dom_display_text extraction.
  • Condition separators for clarity.

Associated samples

Associated hunts

Changes:

- Adding "non-alphabet" condition to .producer FE extraction.
- Adding "security check" condition to .final_dom_display_text extraction.
- Condition separators for clarity.
@missingn0pe
missingn0pe requested a review from a team September 9, 2026 14:12
@missingn0pe
missingn0pe requested a review from a team as a code owner September 9, 2026 14:12
@missingn0pe

Copy link
Copy Markdown
Member Author

Reviewing the original detection need:

It appears these weren't necessarily always court proceedings/summons/etc. There was concern for NLU topic analysis hitting some samples that don't really feel "legal & compliance" in nature (they're likely hitting on disclaimers). Changing confidence creates some misses, unfortunately. This rule tends to reach into other samples that have wide coverage, so I've created a more targeted rule PR for this sample if these changes (or rule as a whole) is not precise enough.

@github-actions github-actions Bot added hunting-required Hunts needed to validate rule efficacy test-rules:excluded:link_analysis Link analysis in rule, excluding from test rules labels Sep 9, 2026
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Test Rules Sync - Excluded

This PR contains rules that use ml.link_analysis, which is not supported in the test-rules environment.

The hunting-required label has been applied. These rules will need to be tested through alternative methods.

github-actions Bot added a commit that referenced this pull request Sep 9, 2026
…themed message or PDF with suspicious indicators
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hunting-required Hunts needed to validate rule efficacy test-rules:excluded:link_analysis Link analysis in rule, excluding from test rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant