Add error message in broker when authorization file not loaded

Author: kleunen

example/broker.cpp

@@ -409,9 +409,15 @@ void run_broker(boost::program_options::variables_map const& vm) {
 
 				std::ifstream input(auth_file);
 
-				MQTT_NS::broker::security security;
-				security.load_json(input);
-				b.set_security(MQTT_NS::force_move(security));
+                if (input) {
+                    MQTT_NS::broker::security security;
+                    security.load_json(input);
+                    b.set_security(MQTT_NS::force_move(security));
+                } else
+                {
+                    MQTT_LOG("mqtt_broker", error)
+                        << "Authorization file '" << auth_file << "' not found,  broker doesn't use authorization file." << std::endl;
+                }
 			}
 		}
 
@@ -651,7 +657,7 @@ int main(int argc, char **argv) {
         std::string config_file = vm["cfg"].as<std::string>();
         if (!config_file.empty()) {
             std::ifstream input(vm["cfg"].as<std::string>());
-            if (input.good()) {
+            if (input) {
                 boost::program_options::store(boost::program_options::parse_config_file(input, desc), vm);
             } else
             {

test/unit/ut_broker_security.cpp

@@ -29,6 +29,7 @@ BOOST_AUTO_TEST_CASE(json_comments) {
     BOOST_CHECK(json_remove_comments("\"#test\"") == "\"#test\"");
     BOOST_CHECK(json_remove_comments("\"'#test'\"") == "\"'#test'\"");
     BOOST_CHECK(json_remove_comments("'\"#test\"'") == "'\"#test\"'");
+    BOOST_CHECK(json_remove_comments("") == "");
 }
 
 BOOST_AUTO_TEST_CASE(default_config) {