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

Feature/epilogue #48

Merged
merged 9 commits into from
Jan 2, 2018
Merged

Feature/epilogue #48

merged 9 commits into from
Jan 2, 2018

Conversation

jhillyerd
Copy link
Owner

Rebase & cherry pick of @pedrinimm contributed PR #41 for issue #32:

Epilogues are now used by spammers or attackers to insert malicious content into emails, this content comes in the form of other mime parts, these come right after the closing boundary of an email.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 85.801% when pulling 12a6e7b on feature/epilogue into bdc4ef5 on develop.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.5%) to 85.789% when pulling a408d9d on feature/epilogue into bdc4ef5 on develop.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.5%) to 85.777% when pulling 0c0d1bc on feature/epilogue into bdc4ef5 on develop.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.5%) to 85.752% when pulling 68e689d on feature/epilogue into bdc4ef5 on develop.

@jhillyerd
Copy link
Owner Author

On my dev machine, average benchmark is quite slow:
BenchmarkSplitEpiloge-4 5000 369788 ns/op 195.05 MB/s

For comparison the quoted-printable cleaner gets 94,898,401 MB/s

@coveralls
Copy link

Coverage Status

Coverage increased (+4.3%) to 90.521% when pulling c5263a1 on feature/epilogue into bdc4ef5 on develop.

@coveralls
Copy link

coveralls commented Jan 2, 2018

Coverage Status

Coverage decreased (-0.08%) to 86.161% when pulling df2387a on feature/epilogue into bdc4ef5 on develop.

@jhillyerd jhillyerd merged commit ffbfc16 into develop Jan 2, 2018
@jhillyerd jhillyerd deleted the feature/epilogue branch January 2, 2018 17:27
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.

2 participants