Merge pull request scality#438 from scality/COMPAT/empty-content-length

COMPAT: PUT empty content-lenght header
cloudberrylab · Dec 3, 2016 · 3d194c3 · 3d194c3
2 parents e688f24 + 5e819f7
commit 3d194c3
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/lib/routes/routePUT.js b/lib/routes/routePUT.js
@@ -24,6 +24,16 @@ export default function routePUT(request, response, log, utapi, statsClient) {
     log.debug('routing request', { method: 'routePUT' });
 
     if (request.objectKey === undefined || request.objectKey === '/') {
+        // PUT bucket - PUT bucket ACL
+
+        // content-length for object is handled separately below
+        const contentLength = request.headers['content-length'];
+        if (contentLength && (isNaN(contentLength) || contentLength === '' ||
+        contentLength < 0)) {
+            log.debug('invalid content-length header');
+            return routesUtils.responseNoBody(
+              errors.BadRequest, null, response, null, log);
+        }
         request.post = '';
         request.on('data', chunk => {
             request.post += chunk.toString();