Ticket #5223 (closed defect: fixed)

Opened 6 years ago

Last modified 6 years ago

Should ignore echoed pubsub notifications

Reported by: gdesmott Owned by: dgilmore
Priority: normal Milestone: Update.1
Component: telepathy-gabble Version:
Keywords: review+ Cc: smcv, daf, robot101, gdesmott
Action Needed: Verified: no
Deployments affected: Blocked By:
Blocking:

Description

While testing sugar using openfire as Jabber server, I noticed Gabble was confused by echoed pubsub notifications making shared activities not appearing on the mesh view. Ejabberd doesn't send these notifications so we didn't notice this before (seems it's an ejabberd bug according to XEP-0163).

Attached a patch (already reviewed and merged upstream) fixing that. I think we should ship it to make sharing work with openfire and avoid regressions if we upgrade our ejabberd server to a version which does send these notifications.

Attachments

gabble-pep-notifications.patch (1.6 kB) - added by gdesmott 6 years ago.

Change History

Changed 6 years ago by gdesmott

Changed 6 years ago by gdesmott

Note that with this patch and the fix of #5224, I can share and play Connect using openfire as server.

Changed 6 years ago by jg

  • milestone changed from Never Assigned to Update.1

Please get this into joyride ASAP for some soaking before it (hopefully) can go into Update.1...

Changed 6 years ago by jg

  • cc gdesmott added

Again, we need to know what needs to be updated.

Changed 6 years ago by gdesmott

This patch fix reached Joyride in telepathy-gabble-0.7.1-0.5.olpc2.i386.rpm. See http://koji.fedoraproject.org/koji/buildinfo?buildID=26876

Changed 6 years ago by jg

  • owner changed from ApprovalForUpdate to dgilmore

Approved.

Changed 6 years ago by gdesmott

  • status changed from new to closed
  • resolution set to fixed

Fixed in telepathy-gabble-0.7.1-0.5.olpc2.i386.rpm which is tagged for Update.1

Note: See TracTickets for help on using tickets.