[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    Re: [Bug 85952] Broken headers in message list with (online) IMAP
From:       Carsten Burghardt <burghardt () kde ! org>
Date:       2004-08-13 16:37:09
Message-ID: 200408131837.12033.burghardt () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Friday 13 August 2004 09:32, Carsten Burghardt wrote:
> Tom Albers sagte:
> > Op donderdag 12 augustus 2004 22:21, schreef Carsten Burghardt:
> >> http://bugs.kde.org/show_bug.cgi?id=85952
> >>
> >> ------- Additional Comments From burghardt kde org  2004-08-12 22:21
> >> ------- CVS commit by burghard:
> >>
> >> Do not remove white spaces from the subject. This is the fix for the
> >> kioslave, the sender/receiver have to be fixed in kmail.
> >> CCMAIL: 85952 bugs kde org
> >>
> >>
> >>   M +1 -1      imapparser.cc   1.87
> >>
> >>
> >> --- kdepim/kioslaves/imap4/imapparser.cc  #1.86:1.87
> >>  @ -679,5 +679,5  @ mailHeader * imapParser::parseEnvelope (
> >>
> >>    //subject
> >> -  envelope->setSubjectEncoded(parseLiteralC(inWords));
> >> +  envelope->setSubject(parseLiteralC(inWords));
> >
> > Carsten,
> >
> > There is an other bug, which is maybe related to this, any chance this
> > bug is
> > resolved as well by this commit?
> >
> > http://bugs.kde.org/show_bug.cgi?id=72310
>
> It's the same bug but both are not fixed yet. This commit only fixes the
> kioslave which was responsible for the subject.
> The problems with the "from" and "to" fields are kmail specific and not
> even imap related. I did a quick test this morning: a sender address like
> "Test,Test <test@test.de>" (without whitespace) is saved correctly in the
> headers (rawHeaderField("from") is correct) but is printed as "Test, Test
> <test@test.de>" by KMMessage::fromStrip(). I think I need some help in
> this field. Ingo, Marc, Don, Till?

The problem is in KMMessage::stripEmailAddr(). If it gets a string like
Test,Test <email>
it generates 
Test, Test <email> 
from it because it thinks that these should be two addresses. This does not 
occur if the string is quoted as it is in the mail ("Test,Test" <email>). So 
there are two ways: either change the kioslave so that it delivers a quoted 
string from the envelope fetch (the server doesn't) or change the 
stripEmailAddr. But what is the bug? Is the described input (without quotes) 
valid?


Carsten

[Attachment #5 (application/pgp-signature)]

_______________________________________________
KMail developers mailing list
KMail-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmail-devel


[prev in list] [next in list] [prev in thread] [next in thread]