json output: remove IP format checking in Config to allow domain names to be entered

Unnecessary IP address value validation step disallowed entering valid domain names/hostnames
as a destination to send JSON records to despite the rest of the plugin code allowing it.

Fixes github issue #91

Author: sedmicha

src/plugins/output/json/src/Config.cpp

@@ -257,21 +257,6 @@ static const struct fds_xml_args args_params[] = {
     FDS_OPTS_END
 };
 
-/**
- * \brief Check if a given string is a valid IPv4/IPv6 address
- * \param[in] ip_addr Address to check
- * \return True or false
- */
-bool
-Config::check_ip(const std::string &ip_addr)
-{
-    in_addr ipv4;
-    in6_addr ipv6;
-
-    return (inet_pton(AF_INET, ip_addr.c_str(), &ipv4) == 1
-        || inet_pton(AF_INET6, ip_addr.c_str(), &ipv6) == 1);
-}
-
 /**
  * \brief Check one of 2 expected options
  *
@@ -436,9 +421,9 @@ Config::parse_send(fds_xml_ctx_t *send)
         throw std::runtime_error("Name of a <send> output must be defined!");
     }
 
-    if (output.addr.empty() || !check_ip(output.addr)) {
+    if (output.addr.empty()) {
         throw std::runtime_error("Value of the element <ip> of the output <send> '" + output.name
-            + "' is not a valid IPv4/IPv6 address");
+            + "' must be defined!");
     }
 
     outputs.sends.push_back(output);

src/plugins/output/json/src/Config.hpp

@@ -177,7 +177,6 @@ struct cfg_syslog : cfg_output {
 /** Parsed configuration of an instance                                                          */
 class Config {
 private:
-    bool check_ip(const std::string &ip_addr);
     bool check_or(const std::string &elem, const char *value, const std::string &val_true,
         const std::string &val_false);
     bool is_syslog_ascii(const std::string &str);