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.

Sign up for GitHub

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

Jump to bottom

Incorrect rendering of "Delivery Status Notification" email messages #5839

Closed
sosnovsky opened this issue Sep 19, 2024 · 0 comments · Fixed by #5844
Closed

Incorrect rendering of "Delivery Status Notification" email messages #5839

sosnovsky opened this issue Sep 19, 2024 · 0 comments · Fixed by #5844
Assignees
@sosnovsky
Labels
actionable bug
Milestone
8.5.8: Encryption...

Comments

@sosnovsky
Copy link
Collaborator

Customer reported an issue with Delivery Status Notification messages received from Google - FlowCrypt marks them as encrypted and tries to decrypt, causing Problem extracting armoted message error.

I checked message structure and seems like this issue is caused by noname attachment with message/global content type.
Need to handle this case and don't try to decrypt plain message.
@sosnovsky sosnovsky added this to the 8.5.8 milestone Sep 19, 2024
@sosnovsky sosnovsky self-assigned this Sep 19, 2024
sosnovsky added a commit that referenced this issue Sep 19, 2024
@sosnovsky
#5839 Skip “message/global” noname attachments
a4964c3
@sosnovsky sosnovsky mentioned this issue Sep 25, 2024
Merged
5 tasks
sosnovsky added a commit that referenced this issue Oct 21, 2024
@sosnovsky
#5839 Fix parsing noname attachment with "message/global" content type (
b89fc54 
#5844)

* #5839 Skip “message/global” noname attachments

* wip

* add ui test

* wip

* wip

* wip

* wip

* rename isHidden to shoudBeHidden
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sosnovsky sosnovsky

Labels
actionable bug
Projects
None yet
Milestone
8.5.8: Encryption keys generation, en...
Development

Successfully merging a pull request may close this issue.

#5839 Fix parsing noname attachment with "message/global" content type FlowCrypt/flowcrypt-browser
1 participant
@sosnovsky