Changeset

8529:257fd322fc9f

net.server_event: Better outgoing connection error handling Same as 67311cda0625. Check for readability. If a socket is readable after initial connection, it likely means an error, so we call the readcallback for that connection to handle it (and ultimately close).
author Matthew Wild <mwild1@gmail.com>
date Fri, 23 Feb 2018 15:33:02 +0000
parents 8528:67311cda0625
children 8530:075df839c110
files net/server_event.lua
diffstat 1 files changed, 3 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/net/server_event.lua	Fri Feb 23 15:30:00 2018 +0000
+++ b/net/server_event.lua	Fri Feb 23 15:33:02 2018 +0000
@@ -105,6 +105,8 @@
 			self:ontimeout()  -- call timeout listener
 			self:_close()
 			debug( "new connection failed. id:", self.id, "error:", self.fatalerror )
+		elseif EV_READWRITE == event then
+			self.readcallback(event);
 		else
 			if plainssl and has_luasec then  -- start ssl session
 				self:starttls(self._sslctx, true)
@@ -116,7 +118,7 @@
 		self.eventconnect = nil
 		return -1
 	end
-	self.eventconnect = addevent( base, self.conn, EV_WRITE, callback, cfg.CONNECT_TIMEOUT )
+	self.eventconnect = addevent( base, self.conn, EV_READWRITE, callback, cfg.CONNECT_TIMEOUT )
 	return true
 end
 function interface_mt:_start_session(call_onconnect) -- new session, for example after startssl