From: Jean Flach <jean-marcel.flach@icinga.com>
Date: Wed, 20 Sep 2017 15:57:14 +0000 (+0200)
Subject: Add Log Warning in case active-stage is empty
X-Git-Tag: v2.8.0~29^2~1
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=287f72b0a0ee76e1ef5052969677c5bbc672d65b;p=icinga2

Add Log Warning in case active-stage is empty

Maybe Critical instead? Throwing an exception seems unnecessary.

refs #3668
---

diff --git a/lib/remote/configpackageutility.cpp b/lib/remote/configpackageutility.cpp
index b7f04a18f..e0b88aa8e 100644
--- a/lib/remote/configpackageutility.cpp
+++ b/lib/remote/configpackageutility.cpp
@@ -254,7 +254,11 @@ String ConfigPackageUtility::GetActiveStage(const String& packageName)
 	fp.close();
 
 	if (fp.fail())
-		return "";
+		stage = "";
+
+	if (stage.IsEmpty())
+		Log(LogWarning, "ConfigPackageUtility")
+			<< "Could not determinate the active stage, does \"" << path << "\" exist?";
 
 	return stage.Trim();
 }