From 9b8f11f517c396df9a26abdbf65435a15fc37810 Mon Sep 17 00:00:00 2001 From: Shane Kilkelly Date: Tue, 1 Sep 2015 12:25:10 +0100 Subject: [PATCH] Fix the check for a success response from S3. We need to handle both a 200 and 206 (partial content) response. --- services/filestore/app/coffee/S3PersistorManager.coffee | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/services/filestore/app/coffee/S3PersistorManager.coffee b/services/filestore/app/coffee/S3PersistorManager.coffee index ede27b20be..15798185a3 100644 --- a/services/filestore/app/coffee/S3PersistorManager.coffee +++ b/services/filestore/app/coffee/S3PersistorManager.coffee @@ -77,8 +77,8 @@ module.exports = if res.statusCode == 404 logger.log bucketName:bucketName, key:key, "file not found in s3" return callback new Errors.NotFoundError("File not found in S3: #{bucketName}:#{key}"), null - if res.stausCode != 200 - logger.log bucketName:bucketName, key:key, "error getting file from s3" + if res.statusCode not in [200, 206] + logger.log bucketName:bucketName, key:key, "error getting file from s3: #{res.statusCode}" return callback new Error("Got non-200 response from S3: #{res.statusCode}"), null callback null, res s3Stream.on 'error', (err) ->