tyler.durden/icinga2
1
0
Fork 0
mirror of https://github.com/Icinga/icinga2.git synced 2025-04-08 17:05:25 +02:00
Code Issues Packages Projects Releases Wiki Activity

IcingaDB#SendRemovedComment(): ignore ack comments like #SendAddedComment()

Browse Source 
Icinga DB doesn't expect comment history for ack comments.

Before:

1. Acked checkable recovers
2. Icinga clears ack comments w/o setting removal time
3. Icinga DB gets neither removal time, nor expire time
4. Icinga DB falls back to NULL and violates NOT NULL constraint
This commit is contained in:
Alexander A. Klimov 2022-03-21 17:06:35 +01:00 committed by Julian Brost
parent f67a5532dc
commit 8d3a847998
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/icingadb/icingadb-objects.cpp
View File

@ -2022,7 +2022,7 @@ void IcingaDB::SendAddedComment(const Comment::Ptr& comment)
void IcingaDB::SendRemovedComment(const Comment::Ptr& comment)
{
if (!GetActive()) {
if (comment->GetEntryType() != CommentUser || !GetActive()) {
return;
}