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 multipart messages #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

koke24
Copy link

@koke24 koke24 commented Nov 22, 2017

Thank you for this software!

I was using dumbster to test multipart messages and I found it didn't handle them right, missing some newline characters. I have written a small change to handle them and maybe you want to insert it in your code.

Best regards

@gerardnico
Copy link

Man I got the same problem. Can we merge this commit ?

@nfalco79
Copy link
Contributor

nfalco79 commented Oct 11, 2021

@kirviq could you provide a quick check and if ok merge it into master and perform a release? Otherwise every test will return a wrong numbers of sent emails
@koke24 could you provide some test 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.

3 participants