bucket notifications - allow empty notifications content to facilitat…

…e notification conf removal

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

src/endpoint/s3/ops/s3_put_bucket_notification.js

@@ -9,17 +9,17 @@ const S3Error = require('../s3_errors').S3Error;
 async function put_bucket_notification(req) {
 
     const topic_configuration = req.body.NotificationConfiguration?.TopicConfiguration;
-    if (!topic_configuration ||
-        typeof topic_configuration !== 'object') throw new S3Error(S3Error.MalformedXML);
 
     //adapt to db shcema
-    for (const conf of topic_configuration) {
-        conf.id = conf.Id;
-        conf.event = conf.Event;
-        conf.topic = conf.Topic;
-        delete conf.Id;
-        delete conf.Event;
-        delete conf.Topic;
+    if (topic_configuration) {
+        for (const conf of topic_configuration) {
+            conf.id = conf.Id;
+            conf.event = conf.Event;
+            conf.topic = conf.Topic;
+            delete conf.Id;
+            delete conf.Event;
+            delete conf.Topic;
+        }
     }
 
     const reply = await req.object_sdk.put_bucket_notification({

src/util/notifications_util.js

@@ -301,6 +301,9 @@ function get_connection(connect) {
 
 
 async function test_notifications(bucket, connect_files_dir) {
+    if (!bucket.notifications) {
+        return;
+    }
     const notificator = new Notificator({connect_files_dir});
     for (const notif of bucket.notifications) {
         const connect = await notificator.parse_connect_file(notif.connect);