| Summary: | Kmail (Framework branch) inserts an additional To: and Subject: into the header upon sending emails. | ||
|---|---|---|---|
| Product: | [Applications] kmail2 | Reporter: | Raymond Wooninck <tittiatcoke> |
| Component: | general | Assignee: | kdepim bugs <kdepim-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | major | CC: | dan, montel |
| Priority: | NOR | ||
| Version: | Git (master) | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/kdepimlibs/96e025878d2c62339600858046cc90a29cedbda8 | Version Fixed In: | |
| Sentry Crash Report: | |||
| Attachments: | Fix the header problem | ||
|
Description
Raymond Wooninck
2014-12-01 11:29:08 UTC
It looks almost as though it's a problem with Akonadi. I'm having this exact issue, though I also have duplicate From: headers. If I look at the source of the message sitting in the sent mail folder, it only has one set of headers. This leads me to believe that somewhere between the mail transport agent and the server the message is getting duplicated. This could even be related to duplicate signatures reported in bugs further down. This is definitely a Kmail/Akonadi issue though, I can send mail using other MUAs just fine. Another interesting thing of note for me is the subject header. The first header is missing the subject, the second one has it filled in. From: Dan Oriani <dan@redchops.com> Subject: missing subject To: dan.oriani@gmail.com From: Dan Oriani <dan@redchops.com> To: "'dan.oriani@gmail.com'" <dan.oriani@gmail.com> Subject: Look at these headers Date: Sat, 06 Dec 2014 22:16:51 -0500 Alright, tracked this one down, though I don't know why it does it. All I can do is make a super cheap hack to make it work, but I don't want to distribute it. Whatever is sending the URL to SMTPProtocol::put() is sending a malformed string. Instead of "headers=0" which would turn off header generation, it's sending "eaders=0" which isn't recognized by the smtp job creator, and is causing it to create new header information. I'll see if I can track down what is generating the URL and figure out why it's clipping the first character, but I certainly wouldn't be offended if somebody found it before me ;) Created attachment 89878 [details]
Fix the header problem
This patch fixes the parsing of the string passed to the SMTP KIO slave. It appears it was merely a typo, where the function should have read mid(-1) instead of mid(1). In any case, I have tested this on my machine and the headers are no longer duplicated.
Ok I look at it. I will create an unittest for this code and fix it. Thanks for investigate it. Git commit 96e025878d2c62339600858046cc90a29cedbda8 by Montel Laurent. Committed on 09/12/2014 at 09:11. Pushed by mlaurent into branch 'master'. Fix Bug 341457 - Kmail (Framework branch) inserts an additional To: and Subject: into the header upon sending emails. Based on patch from Dan O. Add unittest for it. QUrl query doesn't return "?...." just query. With KUrl it return "?..." => necessary to query().mid(1) Merge remote-tracking branch 'origin/KDE/4.14' Conflicts: kioslave/src/smtp/tests/requesttest.cpp A +1 -1 kioslave/src/smtp/request.cpp [License: GPL (v2) (+Qt exception)] A +34 -1 kioslave/src/smtp/tests/requesttest.cpp [License: GPL (v2)] R +2 -0 kioslave/src/smtp/tests/requesttest.h http://commits.kde.org/kdepimlibs/96e025878d2c62339600858046cc90a29cedbda8 @Dan O. thanks for investigate it. I added unittest and fixed code. |