From b202af98292d6d7b5053e5a934b916153ce89af9 Mon Sep 17 00:00:00 2001 From: Ted Trask Date: Tue, 28 Sep 2010 07:15:36 +0000 Subject: Made major changes to authenticator and roles to improve efficiency. Changed authenticator.lua to request individual users, rather than always loading everyone. Modified get_userinfo and removed get_userinfo_roles and get_userinfo_skin to remove unnecessary cfe processing. This was especially bad when retrieving the cfe options for skins and roles. This does break the interface used by other modules like acf-tinydns. Validation and cfe processing were moved into password-model, where they belong. Added global variables to authenticator and roles libraries so they can reuse data that was already generated. Modified logon-controller to only check for zero users if logon fails. --- app/acf-util/logon-model.lua | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'app/acf-util/logon-model.lua') diff --git a/app/acf-util/logon-model.lua b/app/acf-util/logon-model.lua index d84e5e9..41cc0e0 100644 --- a/app/acf-util/logon-model.lua +++ b/app/acf-util/logon-model.lua @@ -45,8 +45,8 @@ logon = function (self, userid, password, ip_addr, sessiondir, sessiondata) sessiondata.id = session.random_hash(512) local t = authenticator.get_userinfo (self, userid) sessiondata.userinfo = {} - for name,value in pairs(t.value) do - sessiondata.userinfo[name] = value.value + for name,value in pairs(t) do + sessiondata.userinfo[name] = value end return cfe({ type="boolean", value=true, label="Logon Success" }) else -- cgit v1.2.3