Skip to content

Commit cedc09f

Browse files
committed
Incorrect statusCode was returned when using range headers
1 parent d9cb811 commit cedc09f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

middleware.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,6 @@ module.exports = function(compiler, options) {
163163
if(-1 == ranges) {
164164
res.setHeader('Content-Range', 'bytes */' + content.length);
165165
res.statusCode = 416;
166-
return content;
167166
}
168167

169168
// valid (syntactically invalid/multiple ranges are treated as a regular response)
@@ -241,7 +240,8 @@ module.exports = function(compiler, options) {
241240
res.setHeader(name, options.headers[name]);
242241
}
243242
}
244-
res.statusCode = 200;
243+
// Express automatically sets the statusCode to 200, but not all servers do (Koa).
244+
res.statusCode = res.statusCode || 200;
245245
if(res.send) res.send(content);
246246
else res.end(content);
247247
}

0 commit comments

Comments
 (0)