eve/dns: don't log warning if dns log version not set

If the DNS log version is not set, we default to v2. This should
not be warning, but better logged at the config level.

A warning will still be logged if the value is set but is not
1 or 2.

src/output-json-dns.c

@@ -510,28 +510,36 @@ static DnsVersion JsonDnsParseVersion(ConfNode *conf)
 
     DnsVersion version = DNS_VERSION_DEFAULT;
     intmax_t config_version;
-    if (ConfGetChildValueInt(conf, "version", &config_version)) {
+    const ConfNode *has_version = ConfNodeLookupChild(conf, "version");
-        switch(config_version) {
+
-            case 1:
+    if (has_version != NULL) {
-                version = DNS_VERSION_1;
+        bool invalid = false;
-                break;
+        if (ConfGetChildValueInt(conf, "version", &config_version)) {
-            case 2:
+            switch(config_version) {
-                version = DNS_VERSION_2;
+                case 1:
-                break;
+                    version = DNS_VERSION_1;
-            default:
+                    break;
-                SCLogWarning(SC_ERR_INVALID_ARGUMENT,
+                case 2:
-                        "invalid eve-log dns version option: %"PRIuMAX", "
+                    version = DNS_VERSION_2;
-                        "forcing it to version %u",
+                    break;
-                        config_version, DNS_VERSION_DEFAULT);
+                default:
-                version = DNS_VERSION_DEFAULT;
+                    invalid = true;
-                break;
+                    break;
+            }
+        } else {
+            invalid = true;
+        }
+        if (invalid) {
+            SCLogWarning(SC_ERR_INVALID_ARGUMENT,
+                    "invalid eve-log dns version option: %s, "
+                    "defaulting to version %u",
+                    has_version->val, version);
         }
     } else {
-        SCLogWarning(SC_ERR_INVALID_ARGUMENT,
+        SCLogConfig("eve-log dns version not set, defaulting to version %u",
-                "eve-log dns version not found, forcing it to version %u",
+                version);
-                DNS_VERSION_DEFAULT);
-        version = DNS_VERSION_DEFAULT;
     }
+
     return version;
 }