bucket notifications - fix TopicConfiguration array for get (github i…

…ssue 8647)
noobaa · Jan 7, 2025 · 6939b4a · 6939b4a
1 parent 79b62ac
commit 6939b4a
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/src/endpoint/s3/ops/s3_get_bucket_notification.js b/src/endpoint/s3/ops/s3_get_bucket_notification.js
@@ -14,6 +14,8 @@ async function get_bucket_notification(req) {
 
     result = _.cloneDeep(result);
 
+    const TopicConfiguration = [];
+
     //adapt to aws cli structure
     if (result && result.length > 0) {
         for (const conf of result) {
@@ -23,20 +25,26 @@ async function get_bucket_notification(req) {
             delete conf.event;
             delete conf.topic;
             delete conf.id;
+
+            TopicConfiguration.push({TopicConfiguration: conf});
+        }
+    }
+
+    if (result && result.length > 0) {
+        for (const conf of result) {
+            TopicConfiguration.push({TopicConfiguration: conf});
         }
     }
 
     const reply = result && result.length > 0 ?
         {
             //return result inside TopicConfiguration tag
-            NotificationConfiguration: {
-                TopicConfiguration: result
-            }
+            NotificationConfiguration:
+                TopicConfiguration
         } :
         //if there's no notification, return empty NotificationConfiguration tag
         { NotificationConfiguration: {} };
 
-
     return reply;
 }