Fix issues by httpserver

Co-authored-by: Oliver Kopp <[email protected]>

Author: calixtus

jablib/src/main/java/org/jabref/logic/remote/RemotePreferences.java

@@ -2,13 +2,15 @@
 
 import java.net.InetAddress;
 import java.net.URI;
+import java.net.URISyntaxException;
 import java.net.UnknownHostException;
 
 import javafx.beans.property.BooleanProperty;
 import javafx.beans.property.IntegerProperty;
 import javafx.beans.property.SimpleBooleanProperty;
 import javafx.beans.property.SimpleIntegerProperty;
 
+import org.jspecify.annotations.NonNull;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -75,12 +77,17 @@ public static InetAddress getIpAddress() throws UnknownHostException {
         return InetAddress.getByName("localhost");
     }
 
-    public URI getHttpServerUri() {
+    public @NonNull URI getHttpServerUri() {
         try {
-            return URI.create("http://" + RemotePreferences.getIpAddress().getHostAddress() + ":23119");
-        } catch (UnknownHostException e) {
+            return new URI("http://" + RemotePreferences.getIpAddress().getHostAddress() + ":23119");
+        } catch (UnknownHostException | URISyntaxException e) {
             LOGGER.error("Could not create HTTP server URI. Falling back to default.", e);
-            return URI.create("http://localhost:23119");
+            try {
+                return new URI("http://localhost:23119");
+            } catch (URISyntaxException ex) {
+                LOGGER.error("Should never happen, raw string is already valid uri");
+                throw new RuntimeException(ex);
+            }
         }
     }
 }

jabsrv/src/main/java/org/jabref/http/server/Server.java

@@ -62,9 +62,9 @@ public HttpServer run(List<Path> files, URI uri) {
         // GUI uses HttpServerManager
         Runtime.getRuntime().addShutdownHook(new Thread(() -> {
             try {
-                System.out.println("Shutting down jabsrv...");
+                LOGGER.debug("Shutting down jabsrv...");
                 httpServer.shutdownNow();
-                System.out.println("Done, exit.");
+                LOGGER.debug("Done, exit.");
             } catch (Exception e) {
                 LOGGER.error("Could not shut down server", e);
             }

jabsrv/src/main/java/org/jabref/http/server/cayw/CAYWResource.java

@@ -17,6 +17,7 @@
 
 import javafx.application.Platform;
 
+import org.jabref.architecture.AllowedToUseAwt;
 import org.jabref.http.server.cayw.format.FormatterService;
 import org.jabref.http.server.cayw.gui.CAYWEntry;
 import org.jabref.http.server.cayw.gui.SearchDialog;
@@ -40,6 +41,7 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+@AllowedToUseAwt("Requires ava.awt.datatransfer.Clipboard")
 @Path("better-bibtex/cayw")
 public class CAYWResource {
     private static final Logger LOGGER = LoggerFactory.getLogger(CAYWResource.class);