From 974001196d57ff259356a6dcba75377cbb68a46e Mon Sep 17 00:00:00 2001
From: Yann Leboulanger <asterix@lagaule.org>
Date: Fri, 9 Apr 2010 15:11:04 +0200
Subject: [PATCH] [Gabriel] add <confirm> tag when we reply to XEP-0070 request
 when it's a message. Fixes #5690

---
 src/common/connection_handlers.py | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/common/connection_handlers.py b/src/common/connection_handlers.py
index 208f1b73e2..eba408ce66 100644
--- a/src/common/connection_handlers.py
+++ b/src/common/connection_handlers.py
@@ -990,10 +990,14 @@ ConnectionCaps, ConnectionHandlersBase, ConnectionJingle):
         if not self.connection or self.connected < 2:
             return
         if answer == 'yes':
-            self.connection.send(iq_obj.buildReply('result'))
+            confirm = iq_obj.getTag('confirm')
+            reply = iq_obj.buildReply('result')
+            if iq_obj.getType() == 'message':
+                reply.addChild(node=confirm)
+            self.connection.send(reply)
         elif answer == 'no':
             err = common.xmpp.Error(iq_obj,
-                    common.xmpp.protocol.ERR_NOT_AUTHORIZED)
+                common.xmpp.protocol.ERR_NOT_AUTHORIZED)
             self.connection.send(err)
 
     def _HttpAuthCB(self, con, iq_obj):
-- 
GitLab