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

MultipartRelated Start #124

Closed
fenzad opened this issue Mar 31, 2015 · 13 comments
Closed

MultipartRelated Start #124

fenzad opened this issue Mar 31, 2015 · 13 comments
Labels
compatibility Compatibility with existing software enhancement New feature or request

Comments

@fenzad
Copy link

fenzad commented Mar 31, 2015

The code line:

ContentType.Parameters["start"] = "<" + value.ContentId + ">";

is right?
Thunderbird with this parameter with the characters < and > shows an empty message!

@jstedfast
Copy link
Owner

From rfc2387, we have this:

3.4.  Syntax

     related-param   := [ ";" "start" "=" cid ]
                        [ ";" "start-info"  "="
                           ( cid-list / value ) ]
                        [ ";" "type"  "=" type "/" subtype ]
                        ; order independent

     cid-list        := cid cid-list

     cid             := msg-id     ; c.f. [822]

Then, in rfc822, it defines msg-id as:

     msg-id      =  "<" addr-spec ">"            ; Unique message id

So it looks like Thunderbird is broken.

@fenzad
Copy link
Author

fenzad commented Mar 31, 2015

In the rfc 822 i can find this:


3.2. The Start Parameter

The start parameter, if given, is the content-ID of the compound
object's "root". If not present the "root" is the first body part in
the Multipart/Related entity. The "root" is the element the
applications processes first.


So in 99 % of cases (including mine), the parameter is not necessary. If the parameter is not specified the message is shown correctly in Thunderbird.

@jstedfast
Copy link
Owner

Yep, I know it's not necessary. That doesn't make it not a bug in Thunderbird, though :-)

@jstedfast
Copy link
Owner

Can you file a bug on https://bugzilla.mozilla.org about the Thunderbird bug? Thanks.

@fenzad
Copy link
Author

fenzad commented Mar 31, 2015

The bug already exists https://bugzilla.mozilla.org/show_bug.cgi?id=471402
from 2008...how sad :-(

thanks anyway, but in my condition I guess I'll have to edit and recompile the source code of mimekit.

jstedfast added a commit that referenced this issue Mar 31, 2015
@jstedfast
Copy link
Owner

I've added a work-around to not use a start parameter if it is the first part in the multipart. But shame on Thunderbird for not adhering to the specifications.

@fenzad
Copy link
Author

fenzad commented Apr 1, 2015

Fantastico!
You are absolutely right!!! Shame!

There will be a new version for NuGet with this change?

@jstedfast
Copy link
Owner

Probably not until this weekend when I have time to make a release.

@fenzad
Copy link
Author

fenzad commented Apr 1, 2015

Ok ... the weekend is a reasonable time.
I'm happy to wait!
Thank you very much!

@fenzad
Copy link
Author

fenzad commented Apr 9, 2015

Hi...when a new release in nuget?

@jstedfast
Copy link
Owner

Sorry about that, I got distracted this past weekend. There will be a release this weekend for sure.

@fenzad
Copy link
Author

fenzad commented Apr 9, 2015

Thank you very much!

@jstedfast jstedfast added enhancement New feature or request compatibility Compatibility with existing software labels Apr 12, 2015
@jstedfast
Copy link
Owner

I've released MimeKit 1.0.13 with this fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compatibility Compatibility with existing software enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants