fix: close URL stream via try-with-resources in ExtensionUtil

in.close() not in finally; exception during readLine() leaks the underlying HTTP connection; use try-with-resources to guarantee closure on all paths

Signed-off-by: Nico Piel <nico.piel@hotmail.de>

Author: NicoPiel

client/src/com/mirth/connect/client/ui/extensionmanager/ExtensionUtil.java

@@ -34,15 +34,14 @@ public String getURLContents(String address) {
 
         try {
             URL url = new URL(address);
-            BufferedReader in = new BufferedReader(new InputStreamReader(url.openStream()));
-            String str = null;
+            try (BufferedReader in = new BufferedReader(new InputStreamReader(url.openStream()))) {
+                String str;
 
-            while ((str = in.readLine()) != null) {
-                builder.append(str);
-                builder.append("\r\n");
+                while ((str = in.readLine()) != null) {
+                    builder.append(str);
+                    builder.append("\r\n");
+                }
             }
-
-            in.close();
         } catch (Exception e) {
             // could not load page contents
         }