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
F16853438: D16584.id.diff
Wed, Jun 18, 9:45 AM
F16727829: D16584.diff
Tue, Jun 17, 7:13 PM
Unknown Object (File)
Tue, Jun 17, 9:02 AM
Unknown Object (File)
Tue, May 27, 11:30 AM
Unknown Object (File)
May 8 2025, 2:51 AM
Unknown Object (File)
May 5 2025, 12:02 PM
Unknown Object (File)
Apr 23 2025, 1:45 PM
Unknown Object (File)
Apr 15 2025, 3:37 AM
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
Branch
fix-mail-attachments
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 13802
Build 17836: arc lint + arc unit

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/
101 lines

Diff 39916

View Options

externals/mimemailparser/MimeMailParser.class.php

Loading...
Phabricator · Built by Phacility