Abstract connections with "connection listeners" - Added connlistener for xmppclient - SASL/TLS now use a new session:reset_stream() method - main.lua on its way to being a bit neater

author: Matthew Wild <mwild1@gmail.com>
date: Wed, 22 Oct 2008 17:36:21 +0100
parent: 69:5b664c8fef86
child: 120:ef964468f174
--- a/plugins/mod_tls.lua	Wed Oct 22 17:27:40 2008 +0100
+++ b/plugins/mod_tls.lua	Wed Oct 22 17:36:21 2008 +0100
@@ -3,6 +3,8 @@
 local send = require "core.sessionmanager".send_to_session;
 local sm_bind_resource = require "core.sessionmanager".bind_resource;
 
+local sessions = sessions;
+
 local usermanager_validate_credentials = require "core.usermanager".validate_credentials;
 local t_concat, t_insert = table.concat, table.insert;
 local tostring = tostring;
@@ -16,16 +18,15 @@
 add_handler("c2s_unauthed", "starttls", xmlns_starttls,
 		function (session, stanza)
 			if session.conn.starttls then
-				print("Wants to do TLS...");
 				send(session, st.stanza("proceed", { xmlns = xmlns_starttls }));
-				session.connhandler = new_connhandler("xmpp-client", session);
-				session.notopen = true;
-				if session.conn.starttls() then
-					print("Done");
-				else
-					print("Failed");
-				end
-				
+				-- FIXME: I'm commenting the below, not sure why it was necessary
+				-- sessions[session.conn] = nil;
+				session:reset_stream();
+				session.conn.starttls();
+				session.log("info", "TLS negotiation started...");
+			else
+				-- FIXME: What reply?
+				session.log("warn", "Attempt to start TLS, but TLS is not available on this connection");
 			end
 		end);