View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003581 | SOGo | Web Mail | public | 2016-03-14 14:23 | 2016-03-18 17:11 |
Reporter | st41ker | Assigned To | ludovic | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Platform | any | OS | any | OS Version | any |
Product Version | 2.3.8 | ||||
Fixed in Version | 2.3.10 | ||||
Summary | 0003581: 500 HTTP error during encrypted email forwarding | ||||
Description | If you try to forward encrypted email via SoGo web interface ("Forward messages" preference must be set to "Inline") then 500 error appears in browser. SoGo log follows: Mar 14 16:10:27 sogod [1085]: <0x0x7f1024954c18[NGImap4ResponseParser]> ERROR(-[NGImap4ResponseParser _parseData]): got content size If "Forward messages" set to "As attachment" everything works. | ||||
Steps To Reproduce |
| ||||
Tags | No tags attached. | ||||
Please attach to this ticket a test mail in EML format. |
|
Uploaded test email. |
|
I don't have this problem. Try to produce a stacktrace: http://sogo.nu/nc/support/faq/article/how-do-i-debug-sogo-2.html |
|
That is a patched SoGo installation. Patch from http://sogo.nu/bugs/view.php?id=3576 Unfortunately, the server is under production, and used by users. |
|
Problem is a attachment of size 0: Please test attached patch and provide feedback. |
|
I'm not seeing any attaches, except the Test.eml. |
|
0018-3581.patch (911 bytes)
From 359965e827f56eb95c919cc62f58e9637be55495 Mon Sep 17 00:00:00 2001 From: root <root@poldi2.hopto.org> Date: Thu, 17 Mar 2016 10:58:41 +0100 Subject: [PATCH 18/18] 3581 --- SoObjects/Mailer/SOGoDraftObject.m | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/SoObjects/Mailer/SOGoDraftObject.m b/SoObjects/Mailer/SOGoDraftObject.m index 4fadcea..bcc5757 100644 --- a/SoObjects/Mailer/SOGoDraftObject.m +++ b/SoObjects/Mailer/SOGoDraftObject.m @@ -844,7 +844,11 @@ static NSString *userAgent = nil; while ((currentKey = [attachmentKeys nextObject])) { body = [fetch objectForKey: [currentKey lowercaseString]]; - [bodies addObject: [body objectForKey: @"data"]]; + + if (body) + [bodies addObject: [body objectForKey: @"data"]]; + else + [bodies addObject: [[[NSData alloc] init] autorelease]]; } return bodies; -- 2.1.4 |
|
Today I've tested the patch. Error is gone, and everything works as expected. |
|
sogo: v2 f16cc609 2016-03-18 12:54 Details Diff |
(fix) handle empty body data correctly (fixes 0003581) |
Affected Issues 0003581 |
|
mod - SoObjects/Mailer/SOGoDraftObject.m | Diff File | ||
sogo: master 638dbc90 2016-03-18 12:54 Details Diff |
(fix) handle empty body data correctly (fixes 0003581) |
Affected Issues 0003581 |
|
mod - SoObjects/Mailer/SOGoDraftObject.m | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2016-03-14 14:23 | st41ker | New Issue | |
2016-03-14 14:47 | ludovic | Note Added: 0009741 | |
2016-03-14 14:49 | st41ker | File Added: Test.eml | |
2016-03-14 14:49 | st41ker | Note Added: 0009742 | |
2016-03-14 15:28 | ludovic | Note Added: 0009743 | |
2016-03-14 18:04 | st41ker | Note Added: 0009744 | |
2016-03-14 18:05 | st41ker | Note Edited: 0009744 | |
2016-03-17 19:35 | tfu | Note Added: 0009785 | |
2016-03-17 19:42 | st41ker | Note Added: 0009786 | |
2016-03-17 19:54 | tfu | File Added: 0018-3581.patch | |
2016-03-18 13:27 | st41ker | Note Added: 0009790 | |
2016-03-18 16:55 | ludovic | Changeset attached | => sogo v2 f16cc609 |
2016-03-18 16:55 | ludovic | Assigned To | => ludovic |
2016-03-18 16:55 | ludovic | Resolution | open => fixed |
2016-03-18 16:55 | ludovic | Changeset attached | => sogo master 638dbc90 |
2016-03-18 17:11 | ludovic | Status | new => resolved |
2016-03-18 17:11 | ludovic | Fixed in Version | => 2.3.10 |