mod_saslauth: Don't offer bind/session when they aren't authenticated yet :) [thanks albert, again...]
authorMatthew Wild <mwild1@gmail.com>
Fri, 29 May 2009 18:03:48 +0100
changeset 1217 844ef764ef0e
parent 1216 fd8ce71bc72b
child 1218 8e02c10c9e60
mod_saslauth: Don't offer bind/session when they aren't authenticated yet :) [thanks albert, again...]
plugins/mod_saslauth.lua
--- a/plugins/mod_saslauth.lua	Fri May 29 14:33:55 2009 +0100
+++ b/plugins/mod_saslauth.lua	Fri May 29 18:03:48 2009 +0100
@@ -121,7 +121,10 @@
 local xmpp_session_attr = { xmlns='urn:ietf:params:xml:ns:xmpp-session' };
 module:add_event_hook("stream-features", 
 		function (session, features)												
-			if not session.username and ((not secure_auth_only) or session.secure) then
+			if not session.username then
+				if secure_auth_only and not session.secure then
+					return;
+				end
 				features:tag("mechanisms", mechanisms_attr);
 				-- TODO: Provide PLAIN only if TLS is active, this is a SHOULD from the introduction of RFC 4616. This behavior could be overridden via configuration but will issuing a warning or so.
 					if config.get(session.host or "*", "core", "anonymous_login") then