Closed Bug 1202901 Opened 9 years ago Closed 9 years ago

Earlybird 42a2 email text from Outlook hidden by Lightning 4.4a2

Categories

(Calendar :: E-mail based Scheduling (iTIP/iMIP), defect)

Lightning 4.4
x86
Windows 8.1
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mtreloar, Assigned: MakeMyDay)

References

Details

(Keywords: regression)

Attachments

(3 files)

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)
Flags: needinfo?(mtreloar)
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?
Flags: needinfo?(mtreloar)
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?
Flags: needinfo?(mtreloar)
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 makemyday@gmx-topmail.de?
Flags: needinfo?(mtreloar)
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.
Flags: needinfo?(mtreloar)
Sent by mail. My apologies for sending 5 messages when 1 or 2 would do.
Flags: needinfo?(mtreloar)
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)
Keywords: regression
Do we already have a unit test for ltnInvitationUtils.jsm where we could add this testcase? If not, could we create one? :)
Blocks: 533265
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.
Keywords: checkin-needed
Attachment #8658873 - Flags: approval-calendar-aurora?(philipp)
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: 9 years ago
Keywords: checkin-needed
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.
Attachment #8658873 - Flags: approval-calendar-beta?(philipp)
Attachment #8658873 - Flags: approval-calendar-beta?(philipp)
Attachment #8658873 - Flags: approval-calendar-beta+
Attachment #8658873 - Flags: approval-calendar-aurora?(philipp)
Attachment #8658873 - Flags: approval-calendar-aurora+
You need to log in before you can comment on or make changes to this bug.