aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWaqas Hussain <waqas20@gmail.com>2009-03-30 02:30:06 +0500
committerWaqas Hussain <waqas20@gmail.com>2009-03-30 02:30:06 +0500
commitada40203e97eed723e21c2ee862bad56816ce86a (patch)
treee6ee687b82c3067480b13c192d6f3df723a4cab1
parent042b822ab137342d2c79ea9909573158bd4b2ada (diff)
downloadprosody-ada40203e97eed723e21c2ee862bad56816ce86a.tar.gz
prosody-ada40203e97eed723e21c2ee862bad56816ce86a.zip
Fixed: mod_register: Node prepping was not being applied to usernames (part of issue #57)
-rw-r--r--plugins/mod_register.lua5
1 files changed, 3 insertions, 2 deletions
diff --git a/plugins/mod_register.lua b/plugins/mod_register.lua
index f131a60d..c04eca0a 100644
--- a/plugins/mod_register.lua
+++ b/plugins/mod_register.lua
@@ -13,6 +13,7 @@ local usermanager_user_exists = require "core.usermanager".user_exists;
local usermanager_create_user = require "core.usermanager".create_user;
local datamanager_store = require "util.datamanager".store;
local os_time = os.time;
+local nodeprep = require "util.encodings".stringprep.nodeprep;
module:add_feature("jabber:iq:register");
@@ -62,7 +63,7 @@ module:add_iq_handler("c2s", "jabber:iq:register", function (session, stanza)
local password = query:child_with_name("password");
if username and password then
-- FIXME shouldn't use table.concat
- username = table.concat(username);
+ username = nodeprep(table.concat(username));
password = table.concat(password);
if username == session.username then
if usermanager_create_user(username, password, session.host) then -- password change -- TODO is this the right way?
@@ -133,7 +134,7 @@ module:add_iq_handler("c2s_unauthed", "jabber:iq:register", function (session, s
end
end
-- FIXME shouldn't use table.concat
- username = table.concat(username);
+ username = nodeprep(table.concat(username));
password = table.concat(password);
if usermanager_user_exists(username, session.host) then
session.send(st.error_reply(stanza, "cancel", "conflict"));