Skip to content

Commit

Permalink
Update link_credential_phishing_intent_and_other_indicators.yml
Browse files Browse the repository at this point in the history
Falsely returns true if recipient is null. Adding a valid check
  • Loading branch information
morriscode authored Sep 13, 2023
1 parent dec1558 commit c77b714
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,8 @@ source: |
// this is common in link tracking, both for
// benign marketing traffic but also attackers
any(recipients.to,
strings.icontains(..href_url.url, .email.local_part)
.email.domain.valid
and strings.icontains(..href_url.url, .email.local_part)
and strings.icontains(..href_url.url, .email.domain.domain)
)
)
Expand All @@ -290,12 +291,12 @@ source: |
and (
// freemail providers should never be sending this type of email
sender.email.domain.domain in $free_email_providers
// if not freemail, it's suspicious if the sender's root domain
// doesn't match any links in the body
or all(body.links, .href_url.domain.root_domain != sender.email.domain.root_domain)
)
// first-time sender
and (
(
Expand Down

0 comments on commit c77b714

Please sign in to comment.