Revert "Improve service dependency error logging (#703)" (#712)

This reverts commit 611c92b.
CppMicroServices · Aug 3, 2022 · a34977c · a34977c
1 parent 611c92b
commit a34977c
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 174 deletions.
diff --git a/compendium/DeclarativeServices/test/CMakeLists.txt b/compendium/DeclarativeServices/test/CMakeLists.txt
@@ -78,7 +78,6 @@ set(_declarativeservices_tests
   TestRegistrationManager.cpp
   TestSCRBundleExtension.cpp
   TestServiceComponentRuntimeImpl.cpp
-  TestServiceDependencyErrorLogging.cpp
   TestServiceMetadataParserV1.cpp
   TestSetConfiguration.cpp
   TestSingletonComponentConfiguration.cpp

diff --git a/compendium/DeclarativeServices/test/TestServiceDependencyErrorLogging.cpp b/compendium/DeclarativeServices/test/TestServiceDependencyErrorLogging.cpp
diff --git a/framework/src/service/ServiceListeners.cpp b/framework/src/service/ServiceListeners.cpp
@@ -446,24 +446,7 @@ void ServiceListeners::GetMatchingServiceListeners(const ServiceEvent& evt,
       const LDAPExpr& ldapExpr = sse.GetLDAPExpr();
       if (ldapExpr.IsNull() || ldapExpr.Evaluate(props, false)) {
         set.insert(sse);
-      } else if (evt.GetType() == ServiceEvent::SERVICE_REGISTERED) {
-        // get the props key and value for logging
-        std::string propsKeyValues{};
-
-        auto key = props->Keys_unlocked();
-        for (auto k : key) {
-          auto val = props->Value_unlocked(k).ToString();
-          propsKeyValues += k + " : " + val + ", ";
-        }
-
-        const std::string msg =
-          "Service listener matching LDAP filter could "
-          "not be found using LDAP filter: " +
-          sse.GetFilter() + ". Please verify the LDAP filter value in service properties: " + "{" + propsKeyValues + "}";
-
-          coreCtx->logger->Log(
-            cppmicroservices::logservice::SeverityLevel::LOG_DEBUG, msg);
-        }
+      }
     }
 
     // Check the cache