-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Commit
This means dropping support for node 0.8. Fixes part of #1410
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,7 +3,6 @@ node_js: | |
- "iojs" | ||
- "0.12" | ||
- "0.10" | ||
- "0.8" | ||
|
||
env: | ||
global: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,7 +4,6 @@ | |
|
||
var querystring = require('querystring'); | ||
var common = require('./common'); | ||
var pause = require('connect').utils.pause; | ||
|
||
|
||
var findByPath = function(files, path) { | ||
|
@@ -29,10 +28,7 @@ var createSourceFilesMiddleware = function(filesPromise, serveFile, | |
.replace(/^\/absolute/, '') | ||
.replace(/^\/base/, basePath); | ||
|
||
// Need to pause the request because of proxying, see: | ||
// https://groups.google.com/forum/#!topic/q-continuum/xr8znxc_K5E/discussion | ||
// TODO(vojta): remove once we don't care about Node 0.8 | ||
var pausedRequest = pause(request); | ||
request.pause(); | ||
This comment has been minimized.
Sorry, something went wrong.
This comment has been minimized.
Sorry, something went wrong.
dignifiedquire
Author
Member
|
||
|
||
return filesPromise.then(function(files) { | ||
// TODO(vojta): change served to be a map rather then an array | ||
|
@@ -52,7 +48,7 @@ var createSourceFilesMiddleware = function(filesPromise, serveFile, | |
next(); | ||
} | ||
|
||
pausedRequest.resume(); | ||
request.resume(); | ||
}); | ||
}; | ||
}; | ||
|
@dignifiedquire is this still necessary? This is causing strange interactions with asynchronous functionality in lower middleware (serve-static specifically).
Thanks!