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)

Signed-off-by: Amit Prinz Setter <[email protected]>
  • Loading branch information
alphaprinz committed Jan 7, 2025
1 parent 5c4b49b commit b61d0f0
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions src/endpoint/s3/ops/s3_get_bucket_notification.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,29 +10,31 @@ async function get_bucket_notification(req) {
bucket_name: req.params.bucket,
});

const TopicConfiguration = [];

//adapt to aws cli structure
if (result && result.length > 0) {
for (const conf of result) {
conf.Event = conf.event;
conf.Topic = conf.topic;
conf.Id = conf.id;
delete conf.vent;
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 b61d0f0

Please sign in to comment.