Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Thunderbird does not see user online and can't receive message from a contact with a singe quote in its resource

RESOLVED FIXED in Thunderbird 18.0

Status

Thunderbird
Instant Messaging
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Arnaud Abélard, Assigned: florian)

Tracking

16 Branch
Thunderbird 18.0

Thunderbird Tracking Flags

(thunderbird17 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
If a remote contact has a single quote (') in their XMPP resource, Thunderbird's Instant Messaging client will never see it online. If the contact messages the user using Thunderbird, no message will be received.

If the remote user removes the single quote from its resource, it will show as online in Thunderbird's contact list as expected and Thunderbird will receive messages from the remote user.
(Assignee)

Comment 1

5 years ago
Thanks for reporting this.

When Thunderbird receives a presence stanza with a single quote in the resource, this error appears in the Error Console:
Error: TypeError: this._account._parseJID(...) is null
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Hardware: x86 → All
(Assignee)

Comment 2

5 years ago
Created attachment 666518 [details] [diff] [review]
Patch

The XMPP rfc says that the following characters are excluded for the "node" part of the JID:
    #x22 (")
    #x26 (&)
    #x27 (')
    #x2F (/)
    #x3A (:)
    #x3C (<)
    #x3E (>)
    #x40 (@)
http://xmpp.org/rfcs/rfc3920.html#rfc.section.A.5

But doesn't list any prohibited ASCII character for the resource part (http://xmpp.org/rfcs/rfc3920.html#rfc.section.B.5) so I think we shouldn't exclude any character there.

For the domain part, it just says it should be a valid domain name. The only characters that we really have to exclude in that part for our regexp to work are @ and / but I think the list we currently have is reasonable.
Assignee: nobody → florian
Attachment #666518 - Flags: review?(clokep)
Comment on attachment 666518 [details] [diff] [review]
Patch

The change looks good, I wish those regular expressions were a bit more readable, but I don't think there's a reasonable way to do that.
Attachment #666518 - Flags: review?(clokep) → review+
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/comm-central/rev/d54b3307113e
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
(Assignee)

Comment 5

5 years ago
Comment on attachment 666518 [details] [diff] [review]
Patch

[Approval Request Comment]
Low risk, so I think we should take this on comm-aurora for Thunderbird 17.
Attachment #666518 - Flags: approval-comm-aurora?
(Assignee)

Comment 6

5 years ago
http://hg.instantbird.org/instantbird/rev/d9aff99041bb
Attachment #666518 - Flags: approval-comm-aurora? → approval-comm-aurora+
(Assignee)

Comment 7

5 years ago
https://hg.mozilla.org/releases/comm-aurora/rev/cb225ded5901
status-thunderbird17: --- → fixed
You need to log in before you can comment on or make changes to this bug.