Skip to content

Commit 173318c

Browse files
committed
HEAD doesn't require Upload-Offset header
1 parent 197172f commit 173318c

File tree

2 files changed

+0
-15
lines changed

2 files changed

+0
-15
lines changed

lib/handlers/HeadHandler.js

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,6 @@ class HeadHandler extends BaseHandler {
1717
return super.send(res, 404);
1818
}
1919

20-
// The request MUST include a Upload-Offset header
21-
let offset = req.headers['upload-offset'];
22-
offset = parseInt(offset, 10);
23-
if (isNaN(offset) || offset < 0) {
24-
return super.send(res, 403);
25-
}
26-
2720
const file_name = match[1];
2821
return this.store.getOffset(file_name)
2922
.then((stats) => {

test/Test-HeadHandler.js

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -39,14 +39,6 @@ describe('HeadHandler', () => {
3939
done();
4040
});
4141

42-
it('should 403 if no Upload-Offset header ', (done) => {
43-
req.headers = {};
44-
req.url = `${path}/1234`;
45-
handler.send(req, res);
46-
assert.equal(res.statusCode, 403);
47-
done();
48-
});
49-
5042
it('should resolve with the offset', (done) => {
5143
req.headers = { 'upload-offset': 0 };
5244
req.url = `${path}/1234`;

0 commit comments

Comments
 (0)