View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003152 | SOGo | ActiveSync | public | 2015-03-28 13:25 | 2016-04-13 10:07 |
Reporter | zhb | Assigned To | ludovic | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Product Version | 2.2.17 | ||||
Fixed in Version | 2.3.9 | ||||
Summary | 0003152: Sent email breaks mail thread on BlackBerry 10 smartphone | ||||
Description | ) Add email account as EAS on BlackBerry 10, and as IMAP account on IMAP client (e.g. Mac Mail.app) | ||||
Tags | No tags attached. | ||||
Confirmed. |
|
I've provided temp. workaround in 3146 |
|
https://github.com/inverse-inc/sogo/commit/f5607b3e857270f2efdf7c28a629d93b54029d86 |
|
An additional fix was pushed: https://github.com/inverse-inc/sogo/commit/e71a1c9974cc6f0177b50065a173aa37963cc159 |
|
Mail thread still broken. Not sure it's a BlackBerry 10 issue or a SOGo issue, hope SOGo developers can help figure it out. Once we confirm it's a BlackBerry issue, i will try to report this in BlackBerry forum. Steps to reproduce this issue: 1: Send an email with BlackBerry 10 (device: Passport, OS: 10.3.2.2204) to myself. I received this email in seconds, with mail header: Message-ID: <1st@[my_domain]> 2: Reply to this first received email, received in seconds with below mail headers: Message-ID: <2nd@[my_domain]> So far so good, emails are grouped in ONE thread. 3: Reply to the latest reply, received in seconds with below mail headers: Message-ID: <3rd@[my_domain]> This reply is grouped in a new thread. The working solution is: *) New reply email should write ALL message-id exist in head 'Message-ID' and 'In-Reply-To' of original mail to 'In-Reply-To' in new email. *) New reply email should write ALL message-id exist in head 'Message-ID' and 'References' of original mail to 'References' in new email. For example: In-Reply-To: <1st@[my_domain]> <2nd@[my_domain]> <3rd@[my_domain]> This way, all MUAs should correctly detect the thread. I searched SOGo github repo, i guess we can improve it in file ActiveSync/SOGoActiveSyncDispatcher.m, line 2606: Write Message-ID and In-Reply-To of old emails to In-Reply-To in new email. Reference: http://cr.yp.to/immhf/thread.html |
|
Update: In-Reply-To should contain only the last message id, but References should contains all. |
|
https://github.com/inverse-inc/sogo/commit/4e7a3ce5fac99efd712168986016295e8e47629a |
|
If no References: header, SOGo should create it and append value of In-Reply-To: and Message-ID: in References. |
|
This issue is still reproducible in SoGo 2.3.1. |
|
Additional fix pushed: https://github.com/inverse-inc/sogo/commit/2bccfcb725e9269e01df11ef3b2c03eba675fc4c Please try the upcoming nightly builds, available in about 12 hours from now. |
|
Still not solved with the latest nightly build. I'm now confused which one causes this issue, SOGo, BlackBerry 10, or MUA ... Will try to figure it out. |
|
Any updates on this? |
|
Still not solved with the latest sogo-2.3.5. |
|
I've an update on the issue. |
|
Dear support, Any update on this issue? Current status is just like @st41ker said: "Only third outgoing email in the thread have no earlier Message IDs, so the thread breaks." |
|
Can you please reproduce the problem with SOGoEASDebugEnabled = YES and upload the sogo.log. I'm not able to reproduce the problem and what to see the difference compared to my env. The log with the cmd=sendmail requests would help to understand the problem. |
|
Here's the log of 'Cmd=SendMail' request, let me know if it's not enough: ############### Mar 10 08:43:20 sogod [1356]: mail.mydomain.com "POST /SOGo/Microsoft-Server-ActiveSync?Cmd=SendMail&DeviceType=BlackBerry&User=zhb%40mydomain.com&DeviceId=BB2AAB1F7E HTTP/1.0" 200 0/961 0.480 - - 64K I docoded the content inside <MIME></MIME> block, here's the mail header part: ################### ################### |
|
Did some more testing with a Gmail account, i think the problem is email sent with mobile device + SOGo doesn't have 'References:' header. Below are headers of sent and received emails in order: 1: Gmail -> Me MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Delivered-To: user@gmail.com MIME-Version: 1.0 |
|
Is it a BB10 issue/bug, or a SOGo issue? |
|
Email sent from SOGo web UI contains 'In-Reply-To:', but no 'References:'. |
|
When you reply from bb10 to gmail do you see a Cmd=SmartReply in sogo.log? |
|
I think that I've found the root cause of the issue. Example: References: <m-id1@dom.tld><m-id2@dom.tld><m-id3@dom.tld> Should be: References: <m-id1@dom.tld><m-id2@dom.tld><m-id3@dom.tld><m-id4@dom.tld> I've tested everything by the following way: send emails and replies until mail thread breaks on my BB. Then, I've just modified mail files directly on the server, so all References: headers became oneliners. And my BB started to show threads correctly. Seems like BB IMAP\POP3 client behaves differently, only BB ActiveSync client have such problem with multiline References: header. |
|
I got "Cmd=SmartReply". So looks like SOGo diesn't add/update "References:" header. |
|
Also, when I've checked EAS debug with working headers and without I found the following: Multiple lines(breaking thread): ConversationId of each mail contains BASE64 text of exact References list. Single lines(working thread): ConversationId of each email contains BASE64 text of only the first emails mail-id. Example debug: Working: <To xmlns="Email:">"Alexander Magliovany" <secondemail@youritcrowd.com></To> <To xmlns="Email:">"Alexander Kalashnikov" <firstemail@yandex.ru></To> <To xmlns="Email:">"Alexander Magliovany" <secondemail@youritcrowd.com></To> <To xmlns="Email:">"Alexander Magliovany" <secondemail@youritcrowd.com></To> Not Working: <To xmlns="Email:">"Alexander Magliovany" <secondemail@youritcrowd.com></To> <To xmlns="Email:">"Alexander Kalashnikov" <firstemail@yandex.ru></To> <To xmlns="Email:">"Alexander Magliovany" <secondemail@youritcrowd.com></To> -- |
|
I've found exact place where the problem is. Accorting to https://tools.ietf.org/html/rfc822#section-3.2 field-body can be "folded" as it stated in https://www.w3.org/Protocols/rfc822/3_Lexical.html if ([reference length] > 0) |
|
I think the problem is caused by sope. While unfolding header fields sope consumes the cr and whitespace instead of only the whitespace. |
|
So, how to fix this? |
|
I sent a patch for review to Ludovic. |
|
Thank you. Also, please, consider the following situation: references: <5693EB11.2020703@youritcrowd.com><EEA40A16E7B7C240B0769E7126EA9DF013A93E7B@XMB102FCNC.rim.net> When references includes m-ids without spaces. |
|
Yes, this is exactly the problem I described above. |
|
If I've understood you correctly, then the following headers in the mail will be parsed correctly after the patch? references: <5693EB11.2020703@youritcrowd.com><EEA40A16E7B7C240B0769E7126EA9DF013A93E7B@XMB102FCNC.rim.net> references: <5693EB11.2020703@youritcrowd.com> <EEA40A16E7B7C240B0769E7126EA9DF013A93E7B@XMB102FCNC.rim.net> references: <5693EB11.2020703@youritcrowd.com> |
|
I'm not sure i understand @st41ker's comment. but "References:" should be allowed to contain multiple message-ids separated by space. Even the RFC document shows the example with multiple message-id this way (search the last "References:" in below rfc document): References: <1234@local.machine.example> <3456@example.net> If you check other MUAs, some of they generate "References:" this way too, and it works fine. e.g. Mac Mail.app. And Gmail has one message-id per line. |
|
@zhb. RFC2822 |
|
@tfu |
|
Dear developers, Any update on this issue? |
|
sogo: v2 bed5175a 2016-03-16 08:55 Details Diff |
(fix) properly split long headers (fixes 0003152) |
Affected Issues 0003152 |
|
mod - ActiveSync/SOGoActiveSyncDispatcher.m | Diff File | ||
mod - ActiveSync/SOGoMailObject+ActiveSync.m | Diff File | ||
sogo: master cd6c5695 2016-03-16 08:55 Details Diff |
(fix) properly split long headers (fixes 0003152) Conflicts: ActiveSync/SOGoMailObject+ActiveSync.m |
Affected Issues 0003152 |
|
mod - ActiveSync/SOGoActiveSyncDispatcher.m | Diff File | ||
mod - ActiveSync/SOGoMailObject+ActiveSync.m | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2015-03-28 13:25 | zhb | New Issue | |
2015-03-29 04:05 | st41ker | Note Added: 0008368 | |
2015-03-29 04:07 | st41ker | Note Edited: 0008368 | |
2015-03-30 05:53 | st41ker | Note Added: 0008372 | |
2015-03-30 13:53 | ludovic | Note Added: 0008377 | |
2015-03-30 13:53 | ludovic | Status | new => resolved |
2015-03-30 13:53 | ludovic | Fixed in Version | => 2.3.0 |
2015-03-30 13:53 | ludovic | Resolution | open => fixed |
2015-03-30 13:53 | ludovic | Assigned To | => ludovic |
2015-05-12 14:46 | ludovic | Note Added: 0008471 | |
2015-06-30 13:28 | zhb | Note Added: 0008680 | |
2015-06-30 13:28 | zhb | Status | resolved => feedback |
2015-06-30 13:28 | zhb | Resolution | fixed => reopened |
2015-06-30 13:30 | zhb | Note Edited: 0008680 | |
2015-06-30 13:35 | zhb | Note Edited: 0008680 | |
2015-06-30 13:38 | zhb | Note Edited: 0008680 | |
2015-07-14 03:27 | zhb | Note Added: 0008731 | |
2015-07-14 03:27 | zhb | Status | feedback => assigned |
2015-07-22 14:28 | ludovic | Note Added: 0008748 | |
2015-07-22 14:28 | ludovic | Status | assigned => resolved |
2015-07-22 14:28 | ludovic | Fixed in Version | 2.3.0 => 2.3.1 |
2015-07-22 14:28 | ludovic | Resolution | reopened => fixed |
2015-07-23 06:00 | zhb | Note Added: 0008762 | |
2015-07-23 06:00 | zhb | Status | resolved => feedback |
2015-07-23 06:00 | zhb | Resolution | fixed => reopened |
2015-07-23 12:37 | zhb | Note Edited: 0008762 | |
2015-07-27 16:29 | st41ker | Note Added: 0008783 | |
2015-09-17 20:30 | ludovic | Note Added: 0008930 | |
2015-09-17 20:30 | ludovic | Status | feedback => resolved |
2015-09-17 20:30 | ludovic | Fixed in Version | 2.3.1 => 2.3.3 |
2015-09-17 20:30 | ludovic | Resolution | reopened => fixed |
2015-10-20 15:11 | zhb | Note Added: 0009015 | |
2015-10-20 15:11 | zhb | Status | resolved => feedback |
2015-10-20 15:11 | zhb | Resolution | fixed => reopened |
2015-10-20 15:17 | zhb | Note Edited: 0009015 | |
2015-10-20 15:36 | zhb | Note Edited: 0009015 | |
2016-01-14 20:42 | ludovic | Note Added: 0009281 | |
2016-01-16 10:57 | zhb | Note Added: 0009290 | |
2016-01-16 10:57 | zhb | Status | feedback => assigned |
2016-01-22 07:49 | st41ker | Note Added: 0009309 | |
2016-03-06 05:48 | zhb | Note Added: 0009639 | |
2016-03-09 21:08 | tfu | Note Added: 0009664 | |
2016-03-10 01:08 | zhb | Note Added: 0009674 | |
2016-03-10 01:12 | zhb | Note Edited: 0009674 | |
2016-03-10 01:28 | zhb | Note Edited: 0009674 | |
2016-03-10 01:36 | zhb | Note Added: 0009675 | |
2016-03-10 01:45 | zhb | Note Added: 0009676 | |
2016-03-10 02:02 | zhb | Note Added: 0009677 | |
2016-03-10 06:46 | tfu | Note Added: 0009679 | |
2016-03-10 08:14 | st41ker | Note Added: 0009680 | |
2016-03-10 08:14 | st41ker | Note Edited: 0009680 | |
2016-03-10 08:26 | st41ker | Note Edited: 0009680 | |
2016-03-10 09:16 | zhb | Note Added: 0009681 | |
2016-03-10 09:35 | st41ker | Note Added: 0009682 | |
2016-03-10 09:40 | st41ker | Note View State: 0009682: private | |
2016-03-10 09:43 | st41ker | Note View State: 0009682: public | |
2016-03-10 10:15 | st41ker | Note Edited: 0009682 | |
2016-03-10 10:23 | st41ker | Note Added: 0009686 | |
2016-03-10 10:37 | tfu | Note Added: 0009687 | |
2016-03-10 10:52 | st41ker | Note Added: 0009688 | |
2016-03-10 11:00 | tfu | Note Added: 0009689 | |
2016-03-10 11:07 | st41ker | Note Added: 0009690 | |
2016-03-10 11:51 | tfu | Note Added: 0009693 | |
2016-03-10 11:58 | st41ker | Note Added: 0009694 | |
2016-03-10 12:43 | zhb | Note Added: 0009695 | |
2016-03-10 12:50 | st41ker | Note Added: 0009696 | |
2016-03-12 07:16 | st41ker | Note Added: 0009719 | |
2016-03-16 07:20 | zhb | Note Added: 0009753 | |
2016-03-16 12:56 | ludovic | Changeset attached | => sogo v2 bed5175a |
2016-03-16 12:56 | ludovic | Status | assigned => resolved |
2016-03-16 12:56 | ludovic | Fixed in Version | 2.3.3 => 2.3.9 |
2016-03-16 12:56 | ludovic | Resolution | reopened => fixed |
2016-03-16 12:58 | ludovic | Changeset attached | => sogo master cd6c5695 |