Merge with 0.5

Sat, 19 Sep 2009 17:42:16 +0100

author
Matthew Wild <mwild1@gmail.com>
date
Sat, 19 Sep 2009 17:42:16 +0100
changeset 1792
407f282f559e
parent 1790
f06688f9b6c6 (current diff)
parent 1791
42d0b68a1efd (diff)
child 1794
218f720af304

Merge with 0.5

core/s2smanager.lua file | annotate | diff | comparison | revisions
--- a/core/s2smanager.lua	Sat Sep 19 11:25:59 2009 +0100
+++ b/core/s2smanager.lua	Sat Sep 19 17:42:16 2009 +0100
@@ -174,7 +174,7 @@
 	if not err then -- This is our first attempt
 		log("debug", "First attempt to connect to %s, starting with SRV lookup...", to_host);
 		host_session.connecting = true;
-		local answer, handle;
+		local handle;
 		handle = adns.lookup(function (answer)
 			handle = nil;
 			host_session.connecting = nil;
@@ -236,6 +236,34 @@
 end
 
 function try_connect(host_session, connect_host, connect_port)
+	host_session.connecting = true;
+	local handle;
+	handle = adns.lookup(function (reply)
+		handle = nil;
+		host_session.connecting = nil;
+		if reply and reply[1] and reply[1].a then
+			log("debug", "DNS reply for %s gives us %s", connect_host, reply[1].a);
+			return make_connect(host_session, reply[1].a, connect_port);
+		else
+			log("debug", "DNS lookup failed to get a response for %s", connect_host);
+			if not attempt_connection(host_session, "name resolution failed") then -- Retry if we can
+				log("debug", "No other records to try for %s - destroying", host_session.to_host);
+				destroy_session(host_session); -- End of the line, we can't
+			end
+		end
+	end, connect_host, "A", "IN");
+
+	-- Set handler for DNS timeout
+	add_task(dns_timeout, function ()
+		if handle then
+			adns.cancel(handle, true);
+		end
+	end);
+		
+	return true;
+end
+
+function make_connect(host_session, connect_host, connect_port)
 	host_session.log("info", "Beginning new connection attempt to %s (%s:%d)", host_session.to_host, connect_host, connect_port);
 	-- Ok, we're going to try to connect
 	

mercurial