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.

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

Added support for embedded message #106

Closed
wants to merge 6 commits into from

Conversation

artemevsin
Copy link

Added support for embedded messages. Useful while working with multiple emails resended in one email.

@marcusorjames
Copy link

Hi, was this branch ever perused, one of my clients is interested in having embedded attachment support.

@artemevsin
Copy link
Author

hi, we use this fork in our project, which is used by 3 our clients so we process about 3000 mails/month and it just works without any problem. But I remember that there was some issues like bad encoding or some bugs related to unexpected structure of mail. I'll be happy if you will try this and I'll glad to help you if you will encounter any issues to improve this feature

@marcusorjames
Copy link

Great thank you. I will have a go and let you know if I have any problems :)

@Slamdunk Slamdunk changed the base branch from master to develop October 2, 2017 10:14
Slamdunk added a commit to Slamdunk/imap that referenced this pull request Oct 2, 2017
@Slamdunk Slamdunk added this to the 1.0 milestone Oct 2, 2017
Slamdunk added a commit that referenced this pull request Oct 2, 2017
* Added support for embedded message

* Refactoring fetching structure

* Adding special way to parse content with base64 encoding

* Fixed the problem that not see some type of the Attachments. Copied from pedrosoares/imap

* Fixed the problem, when message has disposition "attachment" and whole content of this message is content of attachment

* remove unneeded imports

* Refactor #106 code

* Revert forgotten changes

* Use built-in ifdparameters
@Slamdunk
Copy link
Collaborator

Slamdunk commented Oct 2, 2017

Merged in #216 thank you very much!

@Slamdunk Slamdunk closed this Oct 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants