From 6ff524a1253d0a35e7f1d0d0513e5631779e6db9 Mon Sep 17 00:00:00 2001 From: Matthew Wild Date: Sat, 5 Mar 2016 23:37:56 +0000 Subject: [PATCH] util.xmppstream: Remove namespace tracking, it's broken as implemented The code fails to handle namespaces on the initial opening tag (when 'stanza' is nil). Fixing this seems not worth the effort for a feature that is not being used. --- util/xmppstream.lua | 23 ----------------------- 1 file changed, 23 deletions(-) diff --git a/util/xmppstream.lua b/util/xmppstream.lua index 3d97acef..7be63285 100644 --- a/util/xmppstream.lua +++ b/util/xmppstream.lua @@ -196,29 +196,6 @@ local function new_sax_handlers(session, stream_callbacks, cb_handleprogress) end end - if stream_callbacks.track_namespaces then - local namespaces = {} - function xml_handlers:StartNamespaceDecl(prefix, url) - if prefix ~= nil then - namespaces[prefix] = url - end - end - function xml_handlers:EndNamespaceDecl(prefix) - if prefix ~= nil then - namespaces[prefix] = nil - end - end - local old_startelement = xml_handlers.StartElement - function xml_handlers:StartElement(tagname, attr) - old_startelement(self, tagname, attr) - local n = {} - for prefix, url in pairs(namespaces) do - n[prefix] = url - end - stanza.namespaces = n - end - end - local function restricted_handler(parser) cb_error(session, "parse-error", "restricted-xml", "Restricted XML, see RFC 6120 section 11.1."); if not parser.stop or not parser:stop() then -- 2.30.2