CLDSRV-481 Check that putMetadata handles null version properly

lib/routes/routeBackbeat.js

@@ -477,10 +477,13 @@ function putMetadata(request, response, bucketInfo, objMd, log, callback) {
         let versioning = bucketInfo.isVersioningEnabled();
         let isNull = false;
 
+        console.log('>>>>> INITAL versionId!!!', versionId);
+
         if (versionId === 'null') {
             isNull = true;
             // Retrieve the null version id from the object metadata.
             versionId = objMd && objMd.versionId;
+            console.log('OBJ VERSION ID', versionId);
             if (!versionId) {
                 // Set isNull in the object metadata to be written.
                 // Since metadata will generate a versionId for the null version,
@@ -497,6 +500,7 @@ function putMetadata(request, response, bucketInfo, objMd, log, callback) {
                     // and then convert bucket to versioned.
                     // If no new versioned objects are added for given object(s), they look like
                     // standalone master keys.
+                    console.log('HERE!!!');
                     versioning = false;
                 } else {
                     const versioningConf = bucketInfo.getVersioningConfiguration();
@@ -521,21 +525,26 @@ function putMetadata(request, response, bucketInfo, objMd, log, callback) {
         }
 
         const options = {
-            versionId,
+            // versionId,
             isNull,
         };
 
         // NOTE: When 'versioning' is set to true and no 'versionId' is specified,
         // it results in the creation of a "new" version, which also updates the master.
         // NOTE: Since option fields are converted to strings when they're sent to Metadata via the query string,
-        // Metadata interprets the value "false" as if it were true.
+        // Metadata interprets the value "false" as if it were true or undefined as it were an empty string.
         // Therefore, to avoid this confusion, we don't pass the versioning parameter at all if its value is false.
         if (versioning) {
             options.versioning = true;
         }
 
+        if (versionId || versionId === '') {
+            options.versionId = versionId;
+        }
+
         log.trace('putting object version', {
             objectKey: request.objectKey, omVal, options });
+        console.log('PUT METADATA OPTIONS <<<<<<', options);
         return metadata.putObjectMD(bucketName, objectKey, omVal, options, log,
             (err, md) => {
                 if (err) {