Skip to content

Add file_hashes_image_warning_banners.txt - #873

Merged
randypargman merged 3 commits into
mainfrom
Add-file_hashes_image_warning_banners.txt
Sep 8, 2026
Merged

randypargman merged 3 commits into
mainfrom
Add-file_hashes_image_warning_banners.txt

Conversation

@randypargman

Copy link
Copy Markdown
Member

This is a list of sha256 hashes of image files added to email by 3rd party systems to warn recipients.
It can be used to differentiate attachments added by a 3rd party system from attachments that were present in the original email from the sender.

This is a useful differentiation to express detection logic that considers the number of attachments.

…o email by 3rd party systems to warn recipients
@randypargman
randypargman requested a review from a team September 8, 2026 21:50
@randypargman
randypargman requested a review from a team as a code owner September 8, 2026 21:50
Comment thread manifest.json
Comment on lines +99 to +104
"file": "file_hashes_image_warning_banners.txt",
"identifier": "file_hashes_image_warning_banners",
"display_name": "File hashes of images added to messages by 3rd party systems to warn recipients.",
"headers": [
"entry"
]

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.

indentation doesn't match the other entries

Suggested change
"file": "file_hashes_image_warning_banners.txt",
"identifier": "file_hashes_image_warning_banners",
"display_name": "File hashes of images added to messages by 3rd party systems to warn recipients.",
"headers": [
"entry"
]
"file": "file_hashes_image_warning_banners.txt",
"identifier": "file_hashes_image_warning_banners",
"display_name": "File hashes of images added to messages by 3rd party systems to warn recipients.",
"headers": [
"entry"
]

Comment thread manifest.json

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.

might be worth including that it's the sha256 of the file somewhere, not sure where. if it's the list name, or the manifest description.

@zoomequipd zoomequipd 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.

non-blocking feedback provided

@randypargman
randypargman merged commit 47cb182 into main Sep 8, 2026
4 checks passed
@randypargman
randypargman deleted the Add-file_hashes_image_warning_banners.txt branch September 8, 2026 23:05
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.

2 participants