[Pidgin] #11253: Invalid XMPP XHTML

Mon Jan 25 11:11:22 EST 2010

#11253: Invalid XMPP XHTML
-----------------------------------+----------------------------------------
 Reporter:  hildjj                 |     Owner:  deryni
     Type:  defect                 |    Status:  new   
Component:  XMPP                   |   Version:  2.6.5 
 Keywords:  xmpp xhtml html xep71  |  
-----------------------------------+----------------------------------------
 [http://xmpp.org/extensions/xep-0071.html XEP-0071] specifies that
 messages should look like this:

 {{{
 <message>
   <body>hi!</body>
   <html xmlns='http://jabber.org/protocol/xhtml-im'>
     <body xmlns='http://www.w3.org/1999/xhtml'>
       <p>hi!</p>
     </body>
   </html>
 </message>
 }}}

 However, Adium-1.4b17 sends:

 {{{
 <message>
   <body>hi!</body>
   <body xmlns='http://www.w3.org/1999/xhtml'>hi!</body>
 </message>
 }}}

 which is wrong in two ways:
 * No html element around the body
 * no p element around the text

 The first one is absolutely an issue for interoperability; naive clients
 will search for a "body" child of message, and display 2 copies of the
 body.  Good clients will blindly ignore the XHTML, since it isn't in the
 right place.

 The second can cause improper line endings in some older clients, where
 messages run together on the same line.

 This was [http://trac.adium.im/ticket/13363 reported] against Adium
 1.4b13.

 Is it possible that this was caused by the fix to ticket #7390, where the
 protocol was changed instead of the storage mechanism?

-- 
Ticket URL: <http://developer.pidgin.im/ticket/11253>
Pidgin <http://pidgin.im>
Pidgin