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

Add support for attachments without content disposition #70

Merged
merged 2 commits into from
Feb 1, 2015

Conversation

ddeboer
Copy link
Owner

@ddeboer ddeboer commented Jan 23, 2015

Let’s see whether we can reproduce any of the issues with attachments.

Fix #55, #56, #33.

@ddeboer ddeboer changed the title [WIP] Add test for attachments (ref #55) [WIP] Add test for attachments Jan 23, 2015
@ddeboer
Copy link
Owner Author

ddeboer commented Jan 29, 2015

Okay, reproducing with Gmail doesn't work, as Gmail correctly sets the disposition header. Nevertheless, this is a real issue with multiple people reporting on it, so I will fix it soon.

@ddeboer ddeboer changed the title [WIP] Add test for attachments Add support for attachments without content disposition Feb 1, 2015
ddeboer added a commit that referenced this pull request Feb 1, 2015
Add support for attachments without content disposition
@ddeboer ddeboer merged commit 66ee8a9 into master Feb 1, 2015
@ddeboer ddeboer deleted the fix-attachments branch February 1, 2015 22:16
@Slamdunk Slamdunk mentioned this pull request Sep 27, 2017
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.

File Attachments don't show up
1 participant