# HG changeset patch # User Matthew Wild # Date 1274351544 -3600 # Node ID 596303990c7c39f769e6fe9b408b93c32ca6e7d3 # Parent ca149818083d06ec624640f68971cd10033b3de9 usermanager, mod_saslauth: Make account provisioning for Cyrus SASL optional (default: not required) diff -r ca149818083d -r 596303990c7c core/usermanager.lua --- a/core/usermanager.lua Thu May 20 11:13:51 2010 +0100 +++ b/core/usermanager.lua Thu May 20 11:32:24 2010 +0100 @@ -16,6 +16,8 @@ local config = require "core.configmanager"; local hosts = hosts; +local require_provisioning = config.get("*", "core", "cyrus_require_provisioning") or false; + module "usermanager" local function is_cyrus(host) return config.get(host, "core", "sasl_backend") == "cyrus"; end @@ -66,12 +68,12 @@ end function user_exists(username, host) - if is_cyrus(host) then return true; end + if not(require_provisioning) and is_cyrus(host) then return true; end return datamanager.load(username, host, "accounts") ~= nil; -- FIXME also check for empty credentials end function create_user(username, password, host) - if is_cyrus(host) then return nil, "Account creation/modification not available with Cyrus SASL."; end + if not(require_provisioning) and is_cyrus(host) then return nil, "Account creation/modification not available with Cyrus SASL."; end return datamanager.store(username, host, "accounts", {password = password}); end diff -r ca149818083d -r 596303990c7c plugins/mod_saslauth.lua --- a/plugins/mod_saslauth.lua Thu May 20 11:13:51 2010 +0100 +++ b/plugins/mod_saslauth.lua Thu May 20 11:32:24 2010 +0100 @@ -27,6 +27,7 @@ local secure_auth_only = module:get_option("c2s_require_encryption") or module:get_option("require_encryption"); local sasl_backend = module:get_option("sasl_backend") or "builtin"; +local require_provisioning = module:get_option("cyrus_require_provisioning") or false; local log = module._log; @@ -105,9 +106,16 @@ session:reset_stream(); return status, ret, err_msg; end - sm_make_authenticated(session, session.sasl_handler.username); - session.sasl_handler = nil; - session:reset_stream(); + + if not(require_provisioning) or usermanager_user_exists(username, session.host) then + sm_make_authenticated(session, session.sasl_handler.username); + session.sasl_handler = nil; + session:reset_stream(); + else + module:log("warn", "SASL succeeded but we don't have an account provisioned for %s", username); + session.sasl_handler = session.sasl_handler:clean_clone(); + return "failure", "not-authorized", "User authenticated successfully, but not provisioned for XMPP"; + end end return status, ret, err_msg; end