From 0119d748022e9afcd03285870ae2c3985cb105d5 Mon Sep 17 00:00:00 2001 From: Waqas Hussain Date: Sat, 20 Jun 2009 18:18:38 +0500 Subject: stanza_router: Skip prepping 'to' in many common cases - #optimization --- core/stanza_router.lua | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/core/stanza_router.lua b/core/stanza_router.lua index 03035e8e..e5454119 100644 --- a/core/stanza_router.lua +++ b/core/stanza_router.lua @@ -66,17 +66,21 @@ function core_process_stanza(origin, stanza) local from_node, from_host, from_resource; local to_bare, from_bare; if to then - node, host, resource = jid_prepped_split(to); - if not host then - log("warn", "Received stanza with invalid destination JID: %s", to); - if stanza.attr.type ~= "error" and stanza.attr.type ~= "result" then - origin.send(st.error_reply(stanza, "modify", "jid-malformed", "The destination address is invalid: "..to)); + if full_sessions[to] or bare_sessions[to] or hosts[to] then + node, host = jid_split(to); -- TODO only the host is needed, optimize + else + node, host, resource = jid_prepped_split(to); + if not host then + log("warn", "Received stanza with invalid destination JID: %s", to); + if stanza.attr.type ~= "error" and stanza.attr.type ~= "result" then + origin.send(st.error_reply(stanza, "modify", "jid-malformed", "The destination address is invalid: "..to)); + end + return; end - return; + to_bare = node and (node.."@"..host) or host; -- bare JID + if resource then to = to_bare.."/"..resource; else to = to_bare; end + stanza.attr.to = to; end - to_bare = node and (node.."@"..host) or host; -- bare JID - if resource then to = to_bare.."/"..resource; else to = to_bare; end - stanza.attr.to = to; end if from and not origin.full_jid then -- We only stamp the 'from' on c2s stanzas, so we still need to check validity -- cgit v1.2.3