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

multipart/related #229

Closed
mrforsythexeter opened this issue Feb 9, 2018 · 2 comments
Closed

multipart/related #229

mrforsythexeter opened this issue Feb 9, 2018 · 2 comments

Comments

@mrforsythexeter
Copy link

mrforsythexeter commented Feb 9, 2018 •
edited
Loading

Currently the library doesn't appear to handle Content-Type: multipart/related;..

So a message I am trying to process looks like this....

Content-Type: multipart/mixed;
	boundary="_009_VI1PR0302MB26699386E951CE94DC5F0DECAFF30VI1PR0302MB2669_"
MIME-Version: 1.0

--_009_VI1PR0302MB26699386E951CE94DC5F0DECAFF30VI1PR0302MB2669_
Content-Type: multipart/related;
	boundary="_008_VI1PR0302MB26699386E951CE94DC5F0DECAFF30VI1PR0302MB2669_";
	type="text/html"

--_008_VI1PR0302MB26699386E951CE94DC5F0DECAFF30VI1PR0302MB2669_
Content-Type: text/html; charset="iso-8859-1"
Content-ID: <A0192685C2D7804B828B7169A059B2A9@xxxxxxxxxxxxxxxxx.co.uk>
Content-Transfer-Encoding: quoted-printable

The last Boundary has the HTML message. (There are other parts after this.. more attachments). Should the library not attempt to load this into the textHtml? or is this a badly formatted email?

It is in the attachments, however the attachments don't appear to hold the mime type from each part.

With this, I could scan the attachments in an attempt to get the right part, and decode from there.
@commanddotcom commanddotcom mentioned this issue Apr 12, 2018
Closed
@nickl-
Copy link
Contributor

nickl- commented Mar 29, 2019

related #39

@nickl- nickl- mentioned this issue Mar 29, 2019
Merged
@Sebbo94BY
Copy link
Collaborator

Fix has been merged.

@Sebbo94BY Sebbo94BY mentioned this issue May 5, 2019
Merged
Sebbo94BY added a commit that referenced this issue May 5, 2019
@Sebbo94BY
Fixed some issues, improved library and PHPUnit tests
ff84254 
- Updated README
- Move phpunit to require-dev
- Add note about installing dev dependencies in README
- Replaced spaces with tabs
- Added PHPUnit tests for MIME decoding
- Updated formatting of PHPUnit function testParsedDateTimeWithEmptyHeaderDate()
- Issue #209: Function to parse datetime correctly RFC2822
- Issue #280: Added 'Sender' to headers and added additional if-conditions 
- Issue #115: getMail() method returns an object even for nonexistent mail ID
- Issue #273: Added connection check to example
- Issue #227: Added Failed-Recipients to IncomingMailHeader
- Issue #140, #246: Improved exception handling and added PHPUnit test
- Issue #140: Added PHPUnit test for testing ConnectionException
- Issue #140: Improved exception / error handling and improved / added PHPUnit tests
- Issue #154: Added ability to change the imap_search option from SE_UID to SE_FREE and added PHPUnit tests
- Issue #306: Added support for US-ASCII and added ability to disable serverEncoding for searchMailbox()
- Imported missing namespaces to avoid 'unknown class' error messages
- Issue #86: Simplified and improved one replace regex for attachment file names
- Issue #247: Improved grabbing of fromName, fromHost, senderName and senderHost
- Issue #39, #71, #229: Fixed body content gets incorrectly processed as attachments
- Issue #122, #150, #167: Added ability to skip processing of attachments to increase performance, when attachments are not required
- PR #284: Added missing PHPUnit tests
- Issue #122, #150, #167: Lazy load message text and attachments data
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mrforsythexeter @nickl- @Sebbo94BY