Skip to content

Commit

Permalink
bucket notifications - fix TopicConfiguration array for get (github i…
Browse files Browse the repository at this point in the history
…ssue 8647) (#8655)

Signed-off-by: Amit Prinz Setter <[email protected]>
  • Loading branch information
alphaprinz authored Jan 7, 2025
1 parent 37154c6 commit fa24e9b
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions src/endpoint/s3/ops/s3_get_bucket_notification.js
Original file line number Diff line number Diff line change
Expand Up @@ -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) {
Expand All @@ -23,20 +25,20 @@ async function get_bucket_notification(req) {
delete conf.event;
delete conf.topic;
delete conf.id;

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;
}

Expand Down

0 comments on commit fa24e9b

Please sign in to comment.