Closed Bug 1202901 Opened 5 years ago Closed 5 years ago
Earlybird 42a2 email text from Outlook hidden by Lightning 4
21.47 KB, text/plain
21.47 KB, text/plain
2.00 KB, patch
|Details | Diff | Splinter Review|
User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:40.0) Gecko/20100101 Firefox/40.0 Build ID: 20150826023504 Steps to reproduce: Disabled all addons - no bug; Re-enabled Lightning; Bug re-appeared Actual results: An invitation sent from an Outlook user showed meeting info when Lightning diabled; Message body is empty when Lightning is enabled Expected results: In TBird 38 (release version), the message body shows the invitation detail
OS: Unspecified → Windows 8.1
Hardware: Unspecified → x86
Additional info: Addon "Show All Body Parts" shows the message content, and adds 3 attachments, one is an *.ics file. Reverting to View Body as Original HTML reverts to blank message text and removes the attachments
Can you please check whether you have any messages in the error log (ctrl+shift+j) when getting displayed the empty body?
Component: Mail Window Front End → E-mail based Scheduling (iTIP/iMIP)
Product: Thunderbird → Calendar
Version: 42 → Lightning 4.4
No error messages or any other messages. Is there a setting I need to turn "on" other than in the Error Console in order to collect errors/messages in the log? I have the "All" set on in the Error Console?
Please enable calendar.debug.log and calendar.debug.log.verbose in the config editor and try again. Can you please also attach the email in question? And just to be sure: you're expiriencing the problem with Lightning only but no further addons, correct?
Thanks for the tips - novice here. And yes, Lightning as only addon causes the problem. I do not see an "attach" option on this site. May I send you a PM at email@example.com?
There is an add attachment link above. Please choose plain/text type when attaching.
If you want to provide it by mail, please make sure you send it as attachment.
Sent by mail. My apologies for sending 5 messages when 1 or 2 would do.
Attachment #8658787 - Attachment description: Bug1202901 Body as All Body Parts ON - Shows content → Bug1202901 Body as original html
Attachment #8658786 - Attachment description: Bug 1202901 Body as Original HTML (blank) → Bug 1202901 Body as all Body Parts visible
Attachment #8658786 - Attachment mime type: message/rfc822 → text/plain
Attachment #8658787 - Attachment mime type: message/rfc822 → text/plain
Thanks. Have you also checked the error lig after enabling debug mode?
OK. I have set: calendar.debug.log user set boolean true calendar.debug.log.verbose user set boolean true And there are errors now. How do I export the log file? I assume you want it as an attachment here.
When I try to open the attached test case in Thunderbird 43.0a1 (20150909030202) I get the following errors: > Error: TypeError: getIdentityForHeader(...) is null > Source File: chrome://messenger/content/mailWindowOverlay.js Line: 1189 > > Error: not well-formed > Source Code: > Join the meeting<https://www147.livemeeting.com/cc/_XML/cerner/join?id=FSN5MR&role=attend&pw=6D9J%5B%60c> > > Error: [ltnMimeConverter] convertToHTML: SyntaxError: An invalid or illegal string was specified > Source File: file:///[...]/components/lightningTextCalendarConverter.js Line: 85
lightningTextCalendarConverter.js was changed with Bug 533265 in Lightning 4.4, maybe this a regression from this check-in. Maybe problem like not escaping control characters (like "<" or ">") when entering the message content into the new html structure and therefore causing some invalid html syntax.
Status: UNCONFIRMED → NEW
Ever confirmed: true
The error is thrown in function field() in ltnInvitationUtils.jsm when called for "description" field when trying to store the result of linkConverter.scanHTML() to content.innerHTML.
Looks like the problem could be fixed by calling linkConverter.scanTXT(aContentText, Components.interfaces.mozITXTToHTMLConv.kEntities) first to escape "<", ">" and "&" before passing the result to linkConverter.scanHTML().
Thanks Stefan for already looking into this - your proposed solution is the appropriate approach to fix the issue.
Assignee: nobody → makemyday
Status: NEW → ASSIGNED
Attachment #8658873 - Flags: review?(philipp)
Do we already have a unit test for ltnInvitationUtils.jsm where we could add this testcase? If not, could we create one? :)
Actually, we haven't. I'm currently working on another patch modifying functionality of lrnInvitationUtils.jsm and will consider to have a unit test for the same starting with that patch.
Attachment #8658873 - Flags: review?(philipp) → review+
I'd prefer a unit test for this, but I understand that writing it won't be simple so I'll leave that up to you.
I plan to start adding unit tests for ltnbInvitationUtils.jsm with bug 1174511, so I leave this patch without.
https://hg.mozilla.org/comm-central/rev/5e95881b6b2511415a53c349b65c08513979f582 Bug 1202901 - Earlybird 42a2 email text from Outlook hidden by Lightning 4.4a2. r=philipp a=aleth
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 4.6
Comment on attachment 8658873 [details] [diff] [review] FixHtmlConversion-V1.diff After the latest merge, this patch needs to go to beta as well.
Backported to releases/comm-beta changeset b508a45a0ba5
Target Milestone: 4.6 → 4.4
You need to log in before you can comment on or make changes to this bug.