Merge pull request #8986 from alphaprinz/notif_backports_19

nimrod-becker · web-flow · commit e3da17352e38 · 2025-05-06T12:23:48.000+03:00
[backport into 5.19] bucket notifications - use node name in eventSource for NC (#8983)
diff --git a/src/util/notifications_util.js b/src/util/notifications_util.js
@@ -161,7 +161,7 @@ class Notificator {
 
     async handle_failed_notification(notif, failure_append, err) {
         if (this.nc_config_fs) {
-            new NoobaaEvent(NoobaaEvent.NOTIFICATION_FAILED).create_event(notif?.meta?.name, err, err.toString());
+            new NoobaaEvent(NoobaaEvent.NOTIFICATION_FAILED).create_event(notif?.meta?.name, err, err?.toString());
         }
 
         if (notif) {

Original file line number	Diff line number	Diff line change
`@@ -161,7 +161,7 @@ class Notificator {`
`161`	`161`
`162`	`162`	`async handle_failed_notification(notif, failure_append, err) {`
`163`	`163`	`if (this.nc_config_fs) {`
`164`		`- new NoobaaEvent(NoobaaEvent.NOTIFICATION_FAILED).create_event(notif?.meta?.name, err, err.toString());`
	`164`	`+ new NoobaaEvent(NoobaaEvent.NOTIFICATION_FAILED).create_event(notif?.meta?.name, err, err?.toString());`
`165`	`165`	`}`
`166`	`166`
`167`	`167`	`if (notif) {`