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

New Rule (Attachment): PDF with embedded Javascript #646

Closed
wants to merge 6 commits into from

Conversation

aidenmitchell
Copy link
Member

No description provided.

@aidenmitchell
Copy link
Member Author

Running hunts now to ensure no FPs.

@aidenmitchell aidenmitchell marked this pull request as ready for review August 3, 2023 18:00
@aidenmitchell aidenmitchell requested a review from a team August 3, 2023 18:00
@notion-workspace
Copy link

@jkamdjou
Copy link
Member

jkamdjou commented Aug 4, 2023

/update-test-rules

github-actions bot pushed a commit that referenced this pull request Aug 4, 2023
New Rule (Attachment): PDF with embedded Javascript by @aidenmitchell
#646
Source SHA 3090dfb
Triggered by @jkamdjou
@notion-workspace
Copy link

@notion-workspace
Copy link

Remittance Invoice.pdf

@morriscode
Copy link
Member

This has never fired in test-rules. But there are use-cases for it. Merging

@morriscode morriscode enabled auto-merge (squash) November 21, 2023 14:41
@morriscode
Copy link
Member

encountering an issue with CI check - Error: Error getting file content from git history "detection-rules/attachment_eml_suspicious_link_language.yml": fatal: path 'detection-rules/attachment_eml_suspicious_link_language.yml' does not exist in 'ac2f202ee9d04835da4a7f55eee0a0bc4d0c6d1e'

Guessing this is due to the age of this PR. Going to recreate PR.

@morriscode morriscode closed this Nov 21, 2023
auto-merge was automatically disabled November 21, 2023 14:53

Pull request was closed

@morriscode morriscode reopened this Nov 21, 2023
@morriscode morriscode closed this Nov 21, 2023
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.

3 participants