fix: remove broken node archive (#20818)

* fix: remove broken node archive

If archive extraction throws
remove arcive to next time re-download.

* remove archive file in exceptions

In cases where the archive can be corrupt
remove file.

---------

Co-authored-by: Zhe Sun <31067185+ZheSun88@users.noreply.github.com>

Author: caalador

flow-server/src/main/java/com/vaadin/flow/server/frontend/installer/NodeInstaller.java

@@ -493,24 +493,29 @@ private void extractFile(File archive, File destinationDirectory)
                         "The archive file {} is corrupted and will be deleted. "
                                 + "Please run the application again.",
                         archive.getPath());
-                boolean deleted = archive.delete();
-                if (!deleted) {
-                    getLogger().error("Failed to remove archive file {}. "
-                            + "Please remove it manually and run the application.",
-                            archive.getPath());
-                }
+                removeArchiveFile(archive);
                 try {
                     FileUtils.deleteDirectory(destinationDirectory);
                 } catch (IOException ioe) {
                     getLogger().error("Failed to remove target directory '{}'",
                             destinationDirectory, ioe);
                 }
+            } else {
+                removeArchiveFile(archive);
             }
 
             throw e;
         }
     }
 
+    private static void removeArchiveFile(File archive) {
+        if (!archive.delete()) {
+            getLogger().error("Failed to remove archive file {}. "
+                    + "Please remove it manually and run the application.",
+                    archive.getPath());
+        }
+    }
+
     private void downloadFileIfMissing(URI downloadUrl, File destination,
             String userName, String password) throws DownloadException {
         if (!destination.exists()) {
@@ -522,6 +527,7 @@ private void downloadFileIfMissing(URI downloadUrl, File destination,
                     return;
                 } catch (DownloadException e) {
                     if (i == MAX_DOWNLOAD_ATTEMPS - 1) {
+                        removeArchiveFile(destination);
                         throw e;
                     }