The dataType is never null

Author: joegallo

modules/ingest-common/src/main/java/org/elasticsearch/ingest/common/CefParser.java

@@ -422,7 +422,6 @@ private Object convertValueToType(String value, DataType type) {
             case TimestampType -> toTimestamp(value);
             case MACAddressType -> toMACAddress(value);
             case IPType -> toIP(value);
-            case null -> value; // default to string if extension is not defined
         };
     }
 
@@ -559,5 +558,10 @@ public void close() {
         }
     }
 
-    private record ExtensionMapping(String key, DataType dataType, @Nullable String ecsKey) {}
+    private record ExtensionMapping(String key, DataType dataType, @Nullable String ecsKey) {
+        ExtensionMapping {
+            Objects.requireNonNull(key);
+            Objects.requireNonNull(dataType);
+        }
+    }
 }