Remove NetworkConfig's freehand options

llarp/config/config.cpp

@@ -163,13 +163,6 @@ namespace llarp
 
     conf.defineOption<std::string>(
         "network", "strict-connect", false, "", AssignmentAcceptor(m_strictConnect));
-
-    // TODO: make sure this is documented... what does it mean though?
-    conf.addUndeclaredHandler(
-        "network", [&](std::string_view, std::string_view name, std::string_view value) {
-          m_options.emplace(name, value);
-          return true;
-        });
   }
 
   void

llarp/config/config.hpp

@@ -27,8 +27,6 @@ namespace llarp
   using Config_impl_t = llarp::ConfigParser::Config_impl_t;
 
   // TODO: don't use these maps. they're sloppy and difficult to follow
-  using FreehandOptions = std::unordered_multimap<std::string, std::string>;
-
   /// Small struct to gather all parameters needed for config generation to reduce the number of
   /// parameters that need to be passed around.
   struct ConfigGenParameters
@@ -74,7 +72,6 @@ namespace llarp
     std::string m_strictConnect;
     std::string m_ifname;
     std::string m_ifaddr;
-    FreehandOptions m_options;
 
     void
     defineConfigOptions(ConfigDefinition& conf, const ConfigGenParameters& params);