CORE-585 fix number format exception (#1026)

fix number format exception

Author: dvoet

service/src/main/java/org/databiosphere/workspacedataservice/service/DataTypeInferer.java

@@ -47,10 +47,13 @@
 import org.databiosphere.workspacedataservice.shared.model.RecordAttributes;
 import org.databiosphere.workspacedataservice.shared.model.RecordType;
 import org.databiosphere.workspacedataservice.shared.model.attributes.JsonAttribute;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.util.CollectionUtils;
 
 public class DataTypeInferer {
 
+  private static final Logger logger = LoggerFactory.getLogger(DataTypeInferer.class);
   private final ObjectMapper objectMapper;
 
   public DataTypeInferer(ObjectMapper mapper) {
@@ -204,7 +207,13 @@ private JsonNode parseToJsonNode(String val) {
       // boolean values
       // as booleans - e.g. `TRUE`, `tRUe`, or `true` ---> `true`
       return objectMapper.readTree(val.toLowerCase());
-    } catch (JsonProcessingException e) {
+    } catch (JsonProcessingException | NumberFormatException e) {
+      // number format exceptions can occur if the input is scientific notation but too big
+      // e.g. 9623e89508858559
+      return null;
+    } catch (Exception e) {
+      // Catch-all for any other exceptions that may occur during parsing
+      logger.info("unexpected exception while parsing json, proceeding as string", e);
       return null;
     }
   }

service/src/test/java/org/databiosphere/workspacedataservice/service/DataTypeInfererTest.java

@@ -92,6 +92,11 @@ void isValidJson() {
     assertThat(inferer.tryJsonObject(Boolean.TRUE.toString())).isEmpty();
     assertThat(inferer.tryJsonObject("True")).isEmpty();
     assertThat(inferer.tryJsonObject("{\"foo\":\"bar\"}")).isPresent();
+
+    // scientific notation
+    assertThat(inferer.tryJsonObject("3e4")).isEmpty();
+    // scientific notation that's too big
+    assertThat(inferer.tryJsonObject("9623e89508858559")).isEmpty();
   }
 
   @Test