mapper: remove configurable service allowlists
Remove the ability to specify an allow list at runtime. Instead,
introspect any DBus service except those that start with
org.freedesktop. This removes the need for argument parsing code and
complex bitbake metadata.
Skip org.freedesktop because applications (OpenBMC or otherwise) should
access org.freedesktop services directly and not indirectly via the
OpenBMC mapper.
Change-Id: I83038a121580dcde2a2b3b1f994b3066cc9d955f
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
diff --git a/src/processing.hpp b/src/processing.hpp
index d17cacd..3777ff4 100644
--- a/src/processing.hpp
+++ b/src/processing.hpp
@@ -4,14 +4,10 @@
#include "types.hpp"
#include <boost/container/flat_map.hpp>
-#include <boost/container/flat_set.hpp>
#include <cassert>
#include <string>
-/** @brief Define allow list and deny list data structure */
-using AllowDenyList = boost::container::flat_set<std::string>;
-
/** @brief The associations definitions interface */
constexpr const char* assocDefsInterface =
"xyz.openbmc_project.Association.Definitions";
@@ -45,16 +41,14 @@
/** @brief Determine if dbus service is something to monitor
*
- * mapper supports an allowlist concept. If an allowlist is provided as input
- * then only dbus objects matching that list is monitored.
+ * mapper does not monitor all DBus services. needToIntrospect determines
+ * whether or not a service is to be monitored.
*
* @param[in] processName - Dbus service name
- * @param[in] allowList - The allow list
*
* @return True if input processName should be monitored, false otherwise
*/
-bool needToIntrospect(const std::string& processName,
- const AllowDenyList& allowList);
+bool needToIntrospect(const std::string& processName);
/** @brief Handle the removal of an existing name in objmgr data structures
*