Skip to content

Different behavior for types = [Recovery] between User and Notification objects #10211

New issue
New issue
Closed
Bug
#10223
Closed
Different behavior for types = [Recovery] between User and Notification objects#10211
Bug
#10223
Assignees
yhabteab
Labels
area/notificationsNotification eventsbugSomething isn't workingref/IP
@julianbrost

Description

@julianbrost
julianbrost
opened on Oct 31, 2024

One can configure types = [Recovery] in both User and Notification objects. For User objects, there is special handling for this scenario to send recovery notifications regardless of whether a problem notification was sent:

icinga2/lib/icinga/notification.cpp

Lines 431 to 439 in d894792

/* on recovery, check if user was notified before */
if (type == NotificationRecovery) {
if (!notifiedProblemUsers->Contains(userName) && (NotificationProblem & user->GetTypeFilter())) {
Log(LogNotice, "Notification")
<< "Notification object '" << notificationName << "': We did not notify user '" << userName
<< "' (Problem types enabled) for a problem before. Not sending Recovery notification.";
continue;
}
}

However, for Notification objects, there is no such handling and configuring such an object simply never notifies. It would probably be a good idea to make this do the same as for the User object.

ref/IP/56200

Metadata

Metadata

Assignees

Labels

area/notificationsNotification eventsbugSomething isn't workingref/IP

Type

Bug

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions