Comment 7 for bug 194642

Revision history for this message
Michael B. Trausch (mtrausch) wrote : Re: [Bug 194642] Re: name parameter for content-type for attachments should be last

On Fri, 2008-03-07 at 15:29 +0000, hggdh wrote:
> Michael, I am not sure. "Confirmed" was used of old for "this is as
> far as bugsquad can go"; we later went on to "Triaged" meaning that. I
> would keep it how it is now, and see if a sponsor complains...
>
> Also -- it might be a good idea to show the output of your change, and
> state if you checked it against "normal" servers. Upstream will
> certainly ask for this.

Indeed. This is what the header now looks like when re-arranged (I am
pasting these from a 40 MB message I just put together to showcase a
bunch of different types of files.

Content-Type: text/x-java; charset=UTF-8; name=WeeklyPay.java
Content-Type: text/x-csharp; charset=UTF-8; name=fsw.cs
Content-Type: application/x-ms-dos-executable; name=threading.exe
Content-Type: image/png; name=mbt
Content-Type: application/pgp-signature; name=signature.asc
Content-Type: application/pgp-signature; name=UbuntuCodeofConduct-1.0.1.txt.asc
Content-Type: audio/x-vorbis+ogg; name=TWiT0134.ogg

I have been using this patch for several days now and everything appears
to be working just fine; the broken server that I am working around now
silently drops charset MIME header parameters, but no other server I
have sent mail through has done that sort of thing, to my knowledge.

> On my immense ignorance of Evo internals, the patch still looks
> good...
> thanks again.

LOL, you and me both. The good news (and this is praise for the
authors/maintainers of Evolution) is that the source is well organized
enough that I only had to spend probably 30 minutes narrowing down the
code that needed to be modified. I was made quite happy by that.

 --- Mike

--
Michael B. Trausch <email address hidden>
home: 404-592-5746, 1 www.trausch.us
cell: 678-522-7934 im: <email address hidden>, jabber
Ubuntu Unofficial Backports Project: http://backports.trausch.us/