Skip to content

Conversation

@peterdj45
Copy link
Member

@peterdj45 peterdj45 requested a review from a team as a code owner May 16, 2025 21:07
@peterdj45
Copy link
Member Author

/update-test-rules

@peterdj45 peterdj45 added the in-test-rules PR is in our testing suite to collect telemetry label May 16, 2025
github-actions bot pushed a commit that referenced this pull request May 16, 2025
Update cred_theft_suspicious_invoice_w_missing_or_image_only_attachme… by @peterdj45
#2713
Source SHA 60ab4a7
Triggered by @peterdj45
Copy link
Member

@MSAdministrator MSAdministrator left a comment

Choose a reason for hiding this comment

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

@peterdj45 I looked through that hunt and noticed this message was getting flagged as well. Worth looking into I suspect.

https://platform.sublime.security/messages/1992e5847bc5ebc872054c2ac1ddabd93c9eb19649eedf4f705105db403a83b7

github-actions bot added a commit that referenced this pull request Jul 24, 2025
@IndiaAce
Copy link
Member

Hey Peter, wondering if we're ready for a review on this? I saw Josh's comment above I wasn't sure if you'd had a chance to review that yet or not!

@IndiaAce
Copy link
Member

I'm still seeing some greymail/spam matches in test rules - curious if we could add some checks to negate stuff like this... maybe we could use topic analysis to try and tune stuff out?

@morriscode
Copy link
Member

@peterdj45 looks like there's conflicts here, and no changes since Luke pinged above?

@peterdj45
Copy link
Member Author

merged logic into #3543, closing this one out

@peterdj45 peterdj45 closed this Nov 17, 2025
github-actions bot added a commit that referenced this pull request Nov 17, 2025
@peterdj45 peterdj45 deleted the patch-5 branch November 18, 2025 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge in-test-rules PR is in our testing suite to collect telemetry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants