Closed Bug 1317425 Opened 8 years ago Closed 8 years ago

XMPP client fetches vCard-temp in loop if it isn’t set already

Categories

(Chat Core :: XMPP, defect)

Product:
Chat Core
Component:
XMPP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 1298574

People

(Reporter: bugs, Unassigned)

Details

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:49.0) Gecko/20100101 Firefox/49.0
Build ID: 20161021084537

Steps to reproduce:

When looking at the biggest consumers of bandwidth on my XMPP service, I saw that Thunderbird 45.4.0 (latest one at this moment) was sending a loop of self-vCard requests if it couldn’t find it, such as:
Client: <iq id='e991b9a9-4bce-4206-8345-b12acb868e14' type='get'><vCard xmlns='vcard-temp'/></iq>
Server: <iq id='e991b9a9-4bce-4206-8345-b12acb868e14' type='error' to='retracted@my-server/Thunderbird'><error type='cancel'><item-not-found xmlns='urn:ietf:params:xml:ns:xmpp-stanzas'/></error></iq>

This probably happens automatically since many users were doing that.


Actual results:

These stanzas were sent in an infinite loop, probably draining the computer’s battery and giving them a poor experience of Thunderbird.


Expected results:

Thinderbird should have handled the absence of vCard properly and stopped requesting it once it got an answer from the server.
aleth, do you know if this is still an issue in TB 52?
Component: Instant Messaging → XMPP
Flags: needinfo?(aleth)
Product: Thunderbird → Chat Core
Version: 45 Branch → 45
Fixed in bug 1298574, no?
Should be already fixed in Thunderbird 51.

Maybe Emmanuel can confirm using Thunderbird Earlybird?
Status: UNCONFIRMED → RESOLVED
Closed: 8 years ago
Flags: needinfo?(aleth)
Resolution: --- → DUPLICATE
Thanks, I’ll believe you since I don’t really need yet another XMPP client. :)

Would it be possible to backport this fix to the next point release of the 45 branch though, so that users of distributions shipping it (like Debian stable it seems) could benefit from it?
(In reply to Emmanuel Gil Peyrot from comment #4)
> Thanks, I’ll believe you since I don’t really need yet another XMPP client.
> :)
> 
> Would it be possible to backport this fix to the next point release of the
> 45 branch though, so that users of distributions shipping it (like Debian
> stable it seems) could benefit from it?

OK. It's too late for 45.5, but I requested uplift in the duplicate bug.
You need to log in before you can comment on or make changes to this bug.