Avoid using COPY_ATTRIBUTES when copying the jars of fast jar

We need to respect the umask.
This was initially reported here:
keycloak/keycloak#43437

Fixes #50556

Author: gsmet

core/deployment/src/main/java/io/quarkus/deployment/pkg/jar/FastJarBuilder.java

@@ -443,8 +443,9 @@ private static void copyDependency(Set<ArtifactKey> parentFirstArtifacts, Output
                         .setPath(targetPath)
                         .setResolvedDependency(appDep);
                 if (removedFromThisArchive.isEmpty()) {
-                    Files.copy(resolvedDep, targetPath, StandardCopyOption.REPLACE_EXISTING,
-                            StandardCopyOption.COPY_ATTRIBUTES);
+                    // let's not use COPY_ATTRIBUTES to make sure we respect the system umask
+                    Files.copy(resolvedDep, targetPath, StandardCopyOption.REPLACE_EXISTING);
+                    Files.setLastModifiedTime(targetPath, Files.getLastModifiedTime(resolvedDep));
                 } else {
                     // we copy jars for which we remove entries to the same directory
                     // which seems a bit odd to me