Ticket #5018 (closed task: fixed)

Opened 7 years ago

Last modified 7 years ago

reduce gabble's stanza size for IBB packets

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

Description

Ejabberd has a default maximum stanza size of 64k, but gabble (IIRC) assumes 256k packets are acceptable. This should be lowered to aid interoperability with real-world Jabber servers without the same hacks that jabber.laptop.org has (now) got. See #4108 for details.

Attachments

lower-muc-tubes-max-stanza-size.patch (0.7 kB) - added by robot101 7 years ago.
proposed patch

Change History

Changed 7 years ago by robot101

proposed patch

Changed 7 years ago by robot101

  • cc kimquirk added
  • owner changed from gdesmott to robot101
  • status changed from new to assigned

I've added a proposed patch, on the basis that base64 produces 4 bytes of output for 3 bytes of input, it adds a 33% overhead. The maximum stanza size is 64k in ejabberd by default, so I've set the input block size to 45k, which will yield a maximum of 60k after base64 encoding, and allow 4k for the XML encapsulation.

Changed 7 years ago by jg

  • keywords Update.1? removed
  • milestone changed from Never Assigned to Update.1

OK for update.1.

Changed 7 years ago by gdesmott

  • keywords review+ added

+1 for the patch

Changed 7 years ago by robot101

  • owner changed from robot101 to morgs
  • status changed from assigned to new

I've pushed telepathy-gabble-0.7.1-0.6.olpc2 with this patch into koji, should appear in Joyride soon. Morgs, can you check activity sharing works OK with this package then assign to dgilmore for inclusion in Update.1 (given the patch is already approved).

Changed 7 years ago by morgs

  • owner changed from morgs to ApprovalForUpdate

Tested telepathy-gabble-0.7.1-0.6.olpc2 in joyride using write and inserting a picture - works. I see this is triaged to Update.1 but I don't see approval for Update.1, so assigning to ApprovalForUpdate.

Build engineer can assign it back to morgs when it's in Update.1 for testing.

Changed 7 years ago by morgs

  • owner changed from ApprovalForUpdate to dgilmore

Changed 7 years ago by morgs

  • cc morgs added

Changed 7 years ago by gdesmott

Please tag telepathy-gabble 0.7.1-0.6.olpc2 or higher for Update.1 so we can close this bug.

Changed 7 years ago by gdesmott

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

telepathy-gabble-0.7.1-0.7.olpc2 is now in Update.1.

Tested and worked fine with build 658

Note: See TracTickets for help on using tickets.