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 6, 2025
1 parent af4290e commit 86bdfe9
Showing 1 changed file with 10 additions and 5 deletions.
15 changes: 10 additions & 5 deletions src/endpoint/s3/ops/s3_get_bucket_notification.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,23 +16,28 @@ async function get_bucket_notification(req) {
conf.Event = conf.event;
conf.Topic = conf.topic;
conf.Id = conf.id;
delete conf.vent;
delete conf.event;
delete conf.topic;
delete conf.id;
}
}

const TopicConfiguration = [];
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;
}

Expand Down

0 comments on commit 86bdfe9

Please sign in to comment.