bucket notifications - wrong etag, size (#8819)

Signed-off-by: Amit Prinz Setter <[email protected]>

src/endpoint/s3/ops/s3_put_object.js

@@ -65,6 +65,8 @@ async function put_object(req, res) {
     }
     s3_utils.set_encryption_response_headers(req, res, reply.encryption);
 
+    res.size_for_notif = size || reply.size;
+
     if (copy_source) {
         // TODO: This needs to be checked regarding copy between diff namespaces
         // In that case we do not have the copy_source property and just read and upload the stream

src/sdk/namespace_fs.js

@@ -2450,7 +2450,8 @@ class NamespaceFS {
         return {
             etag,
             encryption,
-            version_id
+            version_id,
+            size: stat.size
         };
     }
 

src/util/notifications_util.js

@@ -422,7 +422,7 @@ function compose_notification_req(req, res, bucket, notif_conf) {
     let eTag = res.getHeader('ETag');
     //eslint-disable-next-line
     if (eTag && eTag.startsWith('\"') && eTag.endsWith('\"')) {
-        eTag = eTag.substring(2, eTag.length - 2);
+        eTag = eTag.substring(1, eTag.length - 1);
     }
 
     const event = OP_TO_EVENT[req.op_name];
@@ -442,7 +442,7 @@ function compose_notification_req(req, res, bucket, notif_conf) {
             "x-amz-id-2": req.request_id,
     };
     notif.s3.object.key = req.params.key;
-    notif.s3.object.size = res.getHeader('content-length');
+    notif.s3.object.size = res.size_for_notif;
     notif.s3.object.eTag = eTag;
     notif.s3.object.versionId = res.getHeader('x-amz-version-id');
 
@@ -462,6 +462,8 @@ function compose_notification_req(req, res, bucket, notif_conf) {
         notif.s3.object.sequencer = res.seq;
     }
 
+    delete res.size_for_notif;
+
     return compose_meta(notif, notif_conf, bucket);
 }