Annotate

DEPENDS @ 11592:64cfa396bb84

net.server_epoll: Fix reporting of socket connect timeout If the underlying TCP connection times out before the write timeout kicks in, end up here with err="timeout", which the following code treats as a minor issue. Then, due to epoll apparently returning the EPOLLOUT (writable) event too, we go on and try to write to the socket (commonly stream headers). This fails because the socket is closed, which becomes the error returned up the stack to the rest of Prosody. This also trips the 'onconnect' signal, which has effects on various things, such as the net.connect state machine. Probably undesirable effects. With this, we instead return "connection timeout", like server_event, and destroy the connection handle properly. And then nothing else happens because the connection has been destroyed.
author Kim Alvefur <zash@zash.se>
date Mon, 07 Jun 2021 17:37:14 +0200
parent 8454:90962ef41bf3
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
5406
9903a03b7b3e DEPENDS: Update (LuaSec now has a rock) and add link to /doc/depends
Matthew Wild <mwild1@gmail.com>
parents: 4914
diff changeset
1
9903a03b7b3e DEPENDS: Update (LuaSec now has a rock) and add link to /doc/depends
Matthew Wild <mwild1@gmail.com>
parents: 4914
diff changeset
2 For full information on our dependencies, version requirements, and
8454
90962ef41bf3 DEPENDS: Change link to https
Kim Alvefur <zash@zash.se>
parents: 5406
diff changeset
3 where to find them, see https://prosody.im/doc/depends
5406
9903a03b7b3e DEPENDS: Update (LuaSec now has a rock) and add link to /doc/depends
Matthew Wild <mwild1@gmail.com>
parents: 4914
diff changeset
4
9903a03b7b3e DEPENDS: Update (LuaSec now has a rock) and add link to /doc/depends
Matthew Wild <mwild1@gmail.com>
parents: 4914
diff changeset
5 If you have luarocks available on your platform, install the following:
7
dcc5ac721c20 - Remove some debugging code
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
6
5406
9903a03b7b3e DEPENDS: Update (LuaSec now has a rock) and add link to /doc/depends
Matthew Wild <mwild1@gmail.com>
parents: 4914
diff changeset
7 - luaexpat
9903a03b7b3e DEPENDS: Update (LuaSec now has a rock) and add link to /doc/depends
Matthew Wild <mwild1@gmail.com>
parents: 4914
diff changeset
8 - luasocket
9903a03b7b3e DEPENDS: Update (LuaSec now has a rock) and add link to /doc/depends
Matthew Wild <mwild1@gmail.com>
parents: 4914
diff changeset
9 - luafilesystem
9903a03b7b3e DEPENDS: Update (LuaSec now has a rock) and add link to /doc/depends
Matthew Wild <mwild1@gmail.com>
parents: 4914
diff changeset
10 - luasec
7
dcc5ac721c20 - Remove some debugging code
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
11