fix: Do not log an error if the user aborts an upload (#22908)

Author: Artur-

flow-server/src/main/java/com/vaadin/flow/server/DefaultErrorHandler.java

@@ -18,6 +18,7 @@
 import java.io.EOFException;
 import java.net.SocketException;
 import java.net.SocketTimeoutException;
+import java.util.Collections;
 import java.util.HashSet;
 import java.util.Set;
 
@@ -54,6 +55,13 @@
  */
 public class DefaultErrorHandler implements ErrorHandler {
 
+    public static final Set<String> SOCKET_EXCEPTIONS = Collections
+            .unmodifiableSet(Set.of(SocketException.class.getName(),
+                    SocketTimeoutException.class.getName(),
+                    EOFException.class.getName(),
+                    "org.eclipse.jetty.io.EofException",
+                    "org.apache.catalina.connector.ClientAbortException"));
+
     private final Set<String> ignoredExceptions;
     private final Set<String> routeConfigurationExceptions;
 
@@ -70,11 +78,7 @@ protected DefaultErrorHandler(Set<String> ignoredExceptions,
     }
 
     public DefaultErrorHandler() {
-        this.ignoredExceptions = Set.of(SocketException.class.getName(),
-                SocketTimeoutException.class.getName(),
-                EOFException.class.getName(),
-                "org.eclipse.jetty.io.EofException",
-                "org.apache.catalina.connector.ClientAbortException");
+        this.ignoredExceptions = SOCKET_EXCEPTIONS;
         this.routeConfigurationExceptions = Set.of(
                 AmbiguousRouteConfigurationException.class.getName(),
                 InvalidRouteConfigurationException.class.getName(),

flow-server/src/main/java/com/vaadin/flow/server/communication/TransferUtil.java

@@ -37,6 +37,7 @@
 import com.vaadin.flow.internal.UrlUtil;
 import com.vaadin.flow.internal.streams.UploadCompleteEvent;
 import com.vaadin.flow.internal.streams.UploadStartEvent;
+import com.vaadin.flow.server.DefaultErrorHandler;
 import com.vaadin.flow.server.VaadinRequest;
 import com.vaadin.flow.server.VaadinResponse;
 import com.vaadin.flow.server.VaadinSession;
@@ -204,8 +205,15 @@ public static void handleUpload(UploadHandler handler,
                     .warn("File upload failed.", e);
             handler.responseHandled(new UploadResult(false, response, e));
         } catch (Exception e) {
-            LoggerFactory.getLogger(UploadHandler.class)
-                    .error("Exception during upload", e);
+            if (DefaultErrorHandler.SOCKET_EXCEPTIONS
+                    .contains(e.getClass().getName())) {
+                // Client aborted the upload, no need to log an error
+                LoggerFactory.getLogger(UploadHandler.class)
+                        .debug("Client aborted the upload", e);
+            } else {
+                LoggerFactory.getLogger(UploadHandler.class)
+                        .error("Exception during upload", e);
+            }
             handler.responseHandled(new UploadResult(false, response, e));
         }
     }