From 0b8c755a6be4eb4fd139faaebfc2aa4cb0fbede7 Mon Sep 17 00:00:00 2001
From: Tobias Reisinger <development@serguzim.me>
Date: Mon, 29 Jul 2019 12:12:31 +0200
Subject: [PATCH] fix: wrong loop handling (return->continue)

---
 controllers/api_v1_controllers_discover.cc | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/controllers/api_v1_controllers_discover.cc b/controllers/api_v1_controllers_discover.cc
index 18306fa..6927192 100644
--- a/controllers/api_v1_controllers_discover.cc
+++ b/controllers/api_v1_controllers_discover.cc
@@ -97,15 +97,13 @@ void controllers::post_discover(const HttpRequestPtr &req, std::function<void(co
             std::ifstream answer_payload_stream(answer_payload);
 
             Json::CharReaderBuilder json_reader;
-            std::string errs;
+            std::string errors;
             Json::Value client_info;
 
-            if (!Json::parseFromStream(json_reader, answer_payload_stream, &client_info, &errs))
+            if (!Json::parseFromStream(json_reader, answer_payload_stream, &client_info, &errors))
             {
-                // report to the user the failure and their locations in the document.
-                std::cout  << "Failed to parse configuration\n"
-                           << errs;
-                return;
+                LOG_ERROR << "Failed to parse configuration: " << errors;
+                continue;
             }
 
             free(answer_payload);