Skip to content

Handle inline image attachments#149

Open
bendauphinee wants to merge 5 commits intotedious:masterfrom
cpapdotcom:master
Open

Handle inline image attachments#149
bendauphinee wants to merge 5 commits intotedious:masterfrom
cpapdotcom:master

Conversation

@bendauphinee
Copy link
Copy Markdown

Added detection and handling for inlined images attachments.

Added detection and handling for inlined images attachments.
@tedivm
Copy link
Copy Markdown
Member

tedivm commented Aug 25, 2015

Love the concept, but we'll need working tests and all that before this gets pulled.

@bendauphinee
Copy link
Copy Markdown
Author

After I get this working properly (further testing with weird inlining seems to show some gaps), I'll dig into getting tests set up and committed.

Added extension from MIME type code. Override any other file name
specified with the inline name.
When an email is received with a blank subject, a warning was thrown.
This corrects initialization for these cases.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants