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

Noticed a problem with Yosemite Mail.App and feed2imap and I have a fix #9

Closed
sdenaro opened this issue Mar 17, 2015 · 1 comment
Closed

Comments

@sdenaro
Copy link

sdenaro commented Mar 17, 2015

I upgraded to yosemite and noticed that feeds with embedded images via include-images stopped working. I noticed that some html mail with attachments did work. There is a subtle difference in the content id. Then I put in this simple fix:

In itemtomail.rb

Change this line

          :content_id => cid,

To this

          :content_id => "<#{cid}>",

After that, everything seems to work normally. It also still works in k9 on my Android and in ios8 Mail. Didn't try it on windows. Don't know if you'll have time to fix this, but I'm putting it out in case anyone else has this problem.

@sdenaro sdenaro changed the title Notices a problem with Yosemite Mail.App and feed2imap, have a fix Noticed a problem with Yosemite Mail.App and feed2imap and I have a fix Mar 18, 2015
@terceiro
Copy link
Contributor

sorry, but I can't find any occurrence of content_id in itemtomail.rb, or even in the entire codebase. The only place where I could find contend_id was in an embedded copy of Net::IMAP we had, but which has been removed in commit 64455d1.

I am closing this issue for now, if you still have problems please reopen and provide more details, or open a new iseue.

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

No branches or pull requests

2 participants