In JsonParser.java use try-with-resource for JsonReader jsonReader = Json.createReader(new StringReader(jsonString)); in method remapGeographicCoverage(...)

[rtreacy]

No description provided.

[mreekie]

Sizing:

• If this is a single file, then this should be a 3.
• simple fix.

[qqmyers]

Using the JsonUtil.getJsonObject() or getJsonArray methods would be better than just adding a try with resources.

[pdurbin]

@bencomp are you interested in creating a PR to fix this issue? As of 4e1bc5b this is the line in question:

diff --git a/src/main/java/edu/harvard/iq/dataverse/util/json/JsonParser.java b/src/main/java/edu/harvard/iq/dataverse/util/json/JsonParser.java
index febb785cd9..8f3ffd2671 100644
--- a/src/main/java/edu/harvard/iq/dataverse/util/json/JsonParser.java
+++ b/src/main/java/edu/harvard/iq/dataverse/util/json/JsonParser.java
@@ -682,7 +682,7 @@ public class JsonParser {
         // convert DTO to datasetField so we can back valid values.
         Gson gson = new Gson();
         String jsonString = gson.toJson(geoCoverageDTO);
-        JsonReader jsonReader = Json.createReader(new StringReader(jsonString));
+        JsonReader jsonReader = Json.createReader(new StringReader(jsonString)); // HERE
         JsonObject obj = jsonReader.readObject();
         DatasetField geoCoverageField = parseField(obj);

We have a snapshot from Sonarcloud from a couple weeks ago that also shows it: https://sonarcloud.io/code?id=IQSS_dataverse&selected=IQSS_dataverse%3Asrc%2Fmain%2Fjava%2Fedu%2Fharvard%2Fiq%2Fdataverse%2Futil%2Fjson%2FJsonParser.java&line=685