use content type to decide how to parse bucket quota update query

ISSUE: CLDSRV-818

Author: SylvainSenechal

lib/api/bucketUpdateQuota.js

@@ -22,20 +22,26 @@ function validateBucketQuotaProperty(requestBody, next) {
     return next(null, quotaValue);
 }
 
-function parseRequestBody(requestBody, next) {
-    try {
-        const jsonData = JSON.parse(requestBody);
-        if (typeof jsonData !== 'object') {
-            throw new Error('Invalid JSON');
-        }
-        return next(null, jsonData);
-    } catch {
-        return parseString(requestBody, { explicitArray: false }, (xmlError, xmlData) => {
-            if (xmlError) {
-                return next(errorInstances.InvalidArgument.customizeDescription('Request body must be a JSON object'));
+function parseRequestBody(requestBody, contentType, next) {
+    switch (contentType) {
+        case 'application/xml':
+            return parseString(requestBody, { explicitArray: false }, (xmlError, xmlData) => {
+                if (xmlError) {
+                    return next(errorInstances.InvalidArgument.customizeDescription('Invalid XML format'));
+                }
+                return next(null, xmlData);
+            });
+        case 'application/json':
+        default:
+            try {
+                const jsonData = JSON.parse(requestBody);
+                if (typeof jsonData !== 'object') {
+                    throw new Error('Invalid JSON');
+                }
+                return next(null, jsonData);
+            } catch (err) {
+                return next(errorInstances.InvalidArgument.customizeDescription('Invalid JSON format'));
             }
-            return next(null, xmlData);
-        });
     }
 }
 
@@ -56,7 +62,8 @@ function bucketUpdateQuota(authInfo, request, log, callback) {
                 bucket = b;
                 return next(err, bucket);
             }),
-        (bucket, next) => parseRequestBody(request.post, (err, requestBody) => next(err, bucket, requestBody)),
+        (bucket, next) => parseRequestBody(request.post, request.headers['content-type'], (err, requestBody) =>
+            next(err, bucket, requestBody)),
         (bucket, requestBody, next) => validateBucketQuotaProperty(requestBody, (err, quotaValue) =>
             next(err, bucket, quotaValue)),
         (bucket, quotaValue, next) => {