commit 76d37b6b23d6e29b135d35eb3ca8c26653cff458
author Gaurav Gandhi <gauravgandhi@google.com> Thu Oct 27 01:14:08 2022 +0000
committer Gaurav Gandhi <gauravgandhi@google.com> Thu Oct 27 01:18:20 2022 +0000
tree 48b7522aeabf14e644e3505137d159d962fc289e
parent a4216a593f339dd4ce96bc7ad316b1100b6ab7a2

Fix check of onDeleteType in log_handler builder

its a bug in current check, even for delete type we
are checking for `onOpenType`

Signed-off-by: Gaurav Gandhi <gauravgandhi@google.com>
Change-Id: I8d7b4291d3e2ab9028f3d2f52c1b2ac9e1210b3d

bmc/log-handler/log_handlers_builder.cpp

diff --git a/bmc/log-handler/log_handlers_builder.cpp b/bmc/log-handler/log_handlers_builder.cpp
index 2781534..902b817 100644
--- a/bmc/log-handler/log_handlers_builder.cpp
+++ b/bmc/log-handler/log_handlers_builder.cpp
@@ -91,11 +91,11 @@
 
             const auto& onDelete = a.at("delete");
             const std::string& onDeleteType = onOpen.at("type");
-            if (onOpenType == "systemd")
+            if (onDeleteType == "systemd")
             {
                 pack->onDelete = std::move(buildSystemd(onDelete));
             }
-            else if (onOpenType == "skip")
+            else if (onDeleteType == "skip")
             {
                 pack->onDelete = SkipAction::CreateSkipAction();
             }