lib/s3routes/routes.js

@@ -48,16 +48,12 @@ function checkBucketAndKey(bucketName, objectKey, method, reqQuery,
         return errors.InvalidBucketName;
     }
     if (objectKey !== undefined) {
-        console.log('\n\nIn checkBucketAndKey:', objectKey);
         const result = routesUtils.isValidObjectKey(objectKey,
             blacklistedPrefixes.object);
         if (!result.isValid) {
             log.debug('invalid object key', { objectKey });
-            if (result.invalidPrefix) {
+            return errors.InvalidArgument.customizeDescription('Object key ' +
-                return errors.InvalidArgument.customizeDescription('Object ' +
+            `must not start with "${result.invalidPrefix}".`);
-                `key must not start with "${result.invalidPrefix}".`);
-            }
-            return errors.KeyTooLong;
         }
     }
     if ((reqQuery.partNumber || reqQuery.uploadId)

lib/s3routes/routesUtils.js

@@ -874,9 +874,6 @@ const routesUtils = {
         if (invalidPrefix) {
             return { isValid: false, invalidPrefix };
         }
-        if (objectKey.length > 1024) {
-            return { isValid: false };
-        }
         return { isValid: true };
     },
 

package.json

@@ -3,7 +3,7 @@
   "engines": {
     "node": ">=6.9.5"
   },
-  "version": "8.2.0",
+  "version": "8.0.0",
   "description": "Common utilities for the S3 project components",
   "main": "index.js",
   "repository": {

tests/unit/s3routes/routesUtils/isValidObjectKey.js

@@ -1,23 +0,0 @@
-const assert = require('assert');
-const routesUtils = require('../../../../lib/s3routes/routesUtils.js');
-
-const bannedStr = 'banned';
-const prefixBlacklist = [];
-
-describe('routesUtils.isValidObjectKey', () => {
-    it('should return isValid false if object key name starts with a ' +
-    'blacklisted prefix', () => {
-        const result = routesUtils.isValidObjectKey('bannedkey', [bannedStr]);
-        // return { isValid: false, invalidPrefix };
-        assert.strictEqual(result.isValid, false);
-        assert.strictEqual(result.invalidPrefix, bannedStr);
-    });
-
-    it('should return isValid false if object key name exceeds length of 1024',
-    () => {
-        const keyLength1025 = 'a'.repeat(1025);
-        const result = routesUtils.isValidObjectKey(keyLength1025,
-            prefixBlacklist);
-        assert.strictEqual(result.isValid, false);
-    });
-});