Bug 1224758

Summary: Parse attachments on demand, not on message open
Product: Red Hat Enterprise Linux 7 Reporter: Milan Crha <mcrha>
Component: evolutionAssignee: Milan Crha <mcrha>
Status: CLOSED ERRATA QA Contact: Desktop QE <desktop-qa-list>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 7.1CC: jkoten, mbarnes, tpelka, vrutkovs
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: evolution-3.12.11-15.el7 Doc Type: Bug Fix
Doc Text:
Cause: messages with large (text) attachments took long to open Consequence: large (text) attachments were parsed on message open Fix: parse large (text) attachments only on demand, when requested Result: speeds up message open
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-11-19 07:58:37 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
zipped test message none

Description Milan Crha 2015-05-25 14:36:22 UTC
This comes from an upstream bug report [1].

Emails with large text attachments, like CSV or any similar, could cause Evolution UI freeze when the message was selected to be viewed in the message preview. That was caused by the attachment being parsed and formatted right after the message was opened, even it was set to stay collapsed, thus not viewed by the user. The upstream fix makes the message attachment parse and format only after the user makes the attachment shown explicitly, which makes the Evolution freeze again, but not right after the message open.

This doesn't help with attachments which are marked to be viewed inline, but I believe that most of the text attachments are not marked as such.

[1] https://bugzilla.gnome.org/show_bug.cgi?id=733877

Comment 1 Milan Crha 2015-05-25 16:10:54 UTC
Created attachment 1029526 [details]
zipped test message

Unzip the message and import it to one folder in the Evolution. The unpatched version will cause a long (might be more than a minute, so it's just slowly processing further) message rendering as soon as the message is selected, while the patched version shows the message immediately and causes the same busy-state only after the text attachment is expanded (viewed inline).

Comment 3 Milan Crha 2015-05-29 10:44:36 UTC
Tomas' Popela machine behaves slightly differently than mine, the button to expand the attachment was disable to him for images, due to the underlying element in the DOM structure not being provided by the WebKit yet. The updated patch doesn't rely on WebKit in this regard.

Comment 4 Milan Crha 2015-08-19 14:50:20 UTC
There had been discovered a possible regression in the upstream change, which could cause a crash when viewing mailing list message digest, and even I cannot reproduce it in RHEL7, I can quite reliably in Fedora 22, thus better have the follow-up change included than not.

Comment 7 errata-xmlrpc 2015-11-19 07:58:37 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2015-2226.html