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

Missing body #13

Closed
wants to merge 4 commits into from
Closed

Missing body #13

wants to merge 4 commits into from

Conversation

ts2909
Copy link
Contributor

@ts2909 ts2909 commented Dec 8, 2016

We have a case where we only have the message headers, and not the body. The std library mail.ParseMessage handles this by requiring a separate step to parse the body.

Added handling for this situation along with tests. All existing tests still pass.

@coveralls
Copy link

coveralls commented Dec 8, 2016

Coverage Status

Coverage increased (+0.1%) to 81.405% when pulling 5341e32 on ts2909:topic/missingBody into 05c1f73 on jhillyerd:master.

@coveralls
Copy link

coveralls commented Dec 8, 2016

Coverage Status

Coverage increased (+0.1%) to 81.405% when pulling 04febab on ts2909:topic/missingBody into 05c1f73 on jhillyerd:master.

@jhillyerd
Copy link
Owner

Squashed & manually merged.

@jhillyerd jhillyerd closed this Dec 18, 2016
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