Mail MIME should use Mime Detect or file_get_mimetype() to determine the MIME type of attachments.

CommentFileSizeAuthor
#2 mailmime_1122252-2.patch3.19 KBpillarsdotnet
#1 mailmime-6.x-2.15-images-mime-type.patch1 KBdmitriy.trt

Comments

dmitriy.trt’s picture

dmitriy.trt commented
Version: 7.x-2.x-dev » 6.x-2.15
Status: Active » Needs review
StatusFileSize
new mailmime-6.x-2.15-images-mime-type.patch1 KB

I've wrote patch for 6.x-2.15 to implement this.

pillarsdotnet’s picture

pillarsdotnet commented
Status: Needs work » Needs review
StatusFileSize
new mailmime_1122252-2.patch3.19 KB

Thanks! Applied a modified version of the above patch to 6.x-2.x, 7.x-2.x, and 8.x-2.x devel branches.

pillarsdotnet’s picture

pillarsdotnet commented
Issue summary: View changes

Embarrassing typo.

pillarsdotnet’s picture

pillarsdotnet commented
Status: Needs review » Needs work

Moving back to CNW because I still want to use Mime Detect if it is available.

Status: Needs review » Needs work

Correcting issue status, see #2125263: Issue status not taken from last comment during D7 migration.

salvis’s picture

salvis
he/his
commented
Assigned: pillarsdotnet » Unassigned
Issue summary: View changes
Status: Needs work » Closed (won't fix)

This is not going to happen for D6 anymore, and it won't be an issue for D7 after committing #2526178: Embedded images are attached with wrong MIME types and bad names..

salvis’s picture

salvis
he/his
commented
Status: Closed (won't fix) » Closed (fixed)

Actually it was fixed at the time, just the follow-up is dropped.