Merge pull request #238 from nono/close_sockets

Improve the fix for sockets in CLOSE_WAIT
cozy · Feb 5, 2016 · 28872b3 · 28872b3
2 parents 17eeb3f + 02e5bd6
commit 28872b3
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/server/controllers/applications.coffee b/server/controllers/applications.coffee
@@ -13,13 +13,10 @@ getPathForStaticApp = (appName, path, root, callback) ->
     callback lockedpath(root).join path
 
 forwardRequest = (req, res, errTemplate, next) ->
-    connectionClosed = false
-    req.on 'close', -> connectionClosed = true
-    res.on 'close', -> connectionClosed = true
     appName = req.params.name
     shouldStart = -1 is req.url.indexOf 'socket.io'
     appManager.ensureStarted appName, shouldStart, (err, result) ->
-        if connectionClosed
+        if not res.connection or res.connection.destroyed
             return
         else if err?
             error = new Error err.msg