Page MenuHomePhabricator
Differential D16584

Backport fix from php-mime-mail-parser to fix attachment parsing
ClosedPublic
Actions

Authored by bzerr on Sep 21 2016, 9:51 PM.
Tags
None
Referenced Files
F13060102: D16584.diff
Fri, Apr 19, 5:20 PM
Unknown Object (File)
Thu, Apr 11, 10:29 AM
Unknown Object (File)
Mon, Apr 8, 11:48 PM
Unknown Object (File)
Mon, Apr 8, 11:47 PM
Unknown Object (File)
Fri, Apr 5, 11:49 PM
Unknown Object (File)
Tue, Apr 2, 9:42 AM
Unknown Object (File)
Tue, Apr 2, 9:28 AM
Unknown Object (File)
Sun, Mar 31, 12:28 PM
View All Files
Subscribers
epriestley

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rPaf218564e51a: Backport fix from php-mime-mail-parser to fix attachment parsing
Summary
  • Allow proper parsing of attachments with missing Content-Disposition header
Test Plan
  • Create application email for Maniphest.
  • Send example broken email from Outlook 2007 to that address
    Outlook_Inline.eml38 KBDownload

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bzerr updated this revision to Diff 39916.Sep 21 2016, 9:51 PM
bzerr retitled this revision from to Backport fix from php-mime-mail-parser to fix attachment parsing.
bzerr updated this object.
bzerr edited the test plan for this revision. (Show Details)
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptSep 21 2016, 9:51 PM
Herald added a subscriber: epriestley. · View Herald Transcript
bzerr mentioned this in T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.Sep 21 2016, 9:52 PM
epriestley accepted this revision.Sep 21 2016, 10:02 PM
epriestley added a reviewer: epriestley.
epriestley added inline comments.
externals/mimemailparser/MimeMailParser.class.php
260

Missing some text in this comment ("Some mail clients do not..."), but I'll fix that locally. Thanks!

This revision is now accepted and ready to land.Sep 21 2016, 10:02 PM
Closed by commit rPaf218564e51a: Backport fix from php-mime-mail-parser to fix attachment parsing (authored by bzerr, committed by epriestley). · Explain WhySep 21 2016, 10:04 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
externals/
mimemailparser/
102 lines

Diff 39917

View Options

externals/mimemailparser/MimeMailParser.class.php

Loading...
Phabricator · Built by Phacility