github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-11-21 20:47:08 -05:00
Code Issues Projects Releases Packages Wiki Activity

prevent leak of urlStream on failed downloads

Browse source
This commit is contained in:
Brian Gough 2015-04-29 15:58:28 +01:00
parent bb9944e618
commit 76cd52af70
1 changed files with 10 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
services/clsi/app/coffee/UrlFetcher.coffee
View file

@ -30,6 +30,16 @@ module.exports = UrlFetcher =
if res.statusCode >= 200 and res.statusCode < 300 if res.statusCode >= 200 and res.statusCode < 300
urlStream.pipe(fileStream) urlStream.pipe(fileStream)
else else
logger.error statusCode: res.statusCode, url:url, filePath: filePath, "unexpected status code downloading url to cache"
# https://nodejs.org/api/http.html#http_class_http_clientrequest
# If you add a 'response' event handler, then you must consume
# the data from the response object, either by calling
# response.read() whenever there is a 'readable' event, or by
# adding a 'data' handler, or by calling the .resume()
# method. Until the data is consumed, the 'end' event will not
# fire. Also, until the data is read it will consume memory
# that can eventually lead to a 'process out of memory' error.
urlStream.on 'data', () -> # discard the data
callbackOnce(new Error("URL returned non-success status code: #{res.statusCode} #{url}")) callbackOnce(new Error("URL returned non-success status code: #{res.statusCode} #{url}"))
urlStream.on "error", (error) -> urlStream.on "error", (error) ->