Closed Bug 1659323 Opened 4 years ago Closed 4 years ago

vCard 2.1 attachment parsing broken

Categories

(Thunderbird :: Message Reader UI, defect)

x86_64
Linux
defect

Tracking

(thunderbird_esr68 unaffected, thunderbird_esr78+ fixed)

RESOLVED FIXED
81 Branch
Tracking Status
thunderbird_esr68 --- unaffected
thunderbird_esr78 + fixed

People

(Reporter: public, Assigned: darktrojan)

References

Details

(Keywords: regression, Whiteboard: smoketestb78.2.0-pre)

Attachments

(2 files)

Attached file reproduction.eml

Emails with attached vCards generated by earlier Thunderbird versions are not parsed correctly in Thunderbird 78.2.0-pre build 1, at least on Linux x64. Due to bug 583572 the attachment is thus invisible unless either "Display Attachments Inline" is disabled or the email is displayed as plain text.

STR:
0. Make sure that "Menu bar | View | Message Body As" is "Original HTML" and that "Menu bar | View | Display Attachments Inline" is checked (default on clean profile)

  1. Open the attached eml file

Expected:
The mail contains an attachment, displayed inline

Actual:
The mail does not seem to contain an attachment, error console logs

uncaught exception: ParserError: Invalid parameters in 'email;internet:test@example.com'

Regressed by bug 1639430?

Flags: needinfo?(geoff)
Assignee: nobody → geoff
Status: NEW → ASSIGNED

The code for dealing with v2.1 cards didn't remove the ;internet part, which is useless information anyway. It also didn't account for version:2.1 being lower-case, which it really should've.

Flags: needinfo?(geoff)

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/d170d02c657a
Correctly handle "email;internet:" when parsing vCard 2.1. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
Attachment #9170266 - Flags: approval-comm-esr78?

Comment on attachment 9170266 [details]
Bug 1659323 - Correctly handle "email;internet:" when parsing vCard 2.1. r?mkmelin

[Triage Comment]
Approved for esr78

Attachment #9170266 - Flags: approval-comm-esr78? → approval-comm-esr78+

First of all, I am sorry for my English, but I think the problem is still not solved.
My condition is:
Thunderbird 78.2.2 on Windows 7
Menu: View-> Show attachments inline
Menu: View-> Message body as-> Original HTML

When I receive a message containing vCard attached, by selecting the message in the error console I find myself:
uncaught exception: ParserError: Invalid parameters in 'adr; dom: ;;; MyCity'

I understood the problem is in the "; dom" part of the whole string above, but I don't know how to fix it.

It is also no longer possible to modify the vCard: the changes made are discarded when Thunderbird is reopened.

Sincerely thank you for the work you are doing

(In reply to aa4 from comment #7)
If you have a case where it's not working, please file a new bug, and attach a test case (as .eml)

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: