Set fixed file/dir permissions for zip entries when building an archive

Author: Eng-Fouad

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

@@ -25,6 +25,7 @@
 import org.apache.commons.compress.archivers.zip.DefaultBackingStoreSupplier;
 import org.apache.commons.compress.archivers.zip.ParallelScatterZipCreator;
 import org.apache.commons.compress.archivers.zip.ScatterZipOutputStream;
+import org.apache.commons.compress.archivers.zip.UnixStat;
 import org.apache.commons.compress.archivers.zip.ZipArchiveEntry;
 import org.apache.commons.compress.archivers.zip.ZipArchiveEntryRequest;
 import org.apache.commons.compress.archivers.zip.ZipArchiveOutputStream;
@@ -44,6 +45,9 @@ public class ParallelCommonsCompressArchiveCreator implements ArchiveCreator {
 
     private static final InputStreamSupplier EMPTY_SUPPLIER = () -> new ByteArrayInputStream(new byte[0]);
 
+    private static final int DIR_UNIX_MODE = UnixStat.DIR_FLAG | UnixStat.DEFAULT_DIR_PERM;
+    private static final int FILE_UNIX_MODE = UnixStat.FILE_FLAG | UnixStat.DEFAULT_FILE_PERM;
+
     private final Path archivePath;
     private final FileTime entryTimestamp;
     private final ZipArchiveOutputStream archive;
@@ -84,6 +88,7 @@ public class ParallelCommonsCompressArchiveCreator implements ArchiveCreator {
     public void addManifest(Manifest manifest) throws IOException {
         // we add the manifest directly to the final archive to make sure it is the first element added
         ZipArchiveEntry metaInfArchiveEntry = new ZipArchiveEntry("META-INF/");
+        metaInfArchiveEntry.setUnixMode(DIR_UNIX_MODE);
         normalizeTimestamps(metaInfArchiveEntry);
         archive.putArchiveEntry(metaInfArchiveEntry);
         archive.closeArchiveEntry();
@@ -93,6 +98,7 @@ public void addManifest(Manifest manifest) throws IOException {
         byte[] manifestBytes = baos.toByteArray();
 
         ZipArchiveEntry manifestEntry = new ZipArchiveEntry("META-INF/MANIFEST.MF");
+        manifestEntry.setUnixMode(FILE_UNIX_MODE);
         normalizeTimestamps(manifestEntry);
         manifestEntry.setSize(manifestBytes.length);
         archive.putArchiveEntry(manifestEntry);
@@ -169,15 +175,18 @@ private void addDirectoryEntry(final ZipArchiveEntry zipArchiveEntry) throws IOE
         }
         normalizeTimestamps(zipArchiveEntry);
         zipArchiveEntry.setMethod(compressionMethod);
+        zipArchiveEntry.setUnixMode(DIR_UNIX_MODE);
         directories.addArchiveEntry(ZipArchiveEntryRequest.createZipArchiveEntryRequest(zipArchiveEntry, EMPTY_SUPPLIER));
     }
 
     private void addEntry(final ZipArchiveEntry zipArchiveEntry, final InputStreamSupplier streamSupplier) throws IOException {
         normalizeTimestamps(zipArchiveEntry);
         zipArchiveEntry.setMethod(compressionMethod);
         if (zipArchiveEntry.isDirectory() && !zipArchiveEntry.isUnixSymlink()) {
+            zipArchiveEntry.setUnixMode(DIR_UNIX_MODE);
             directories.addArchiveEntry(ZipArchiveEntryRequest.createZipArchiveEntryRequest(zipArchiveEntry, streamSupplier));
         } else {
+            zipArchiveEntry.setUnixMode(FILE_UNIX_MODE);
             scatterZipCreator.addArchiveEntry(zipArchiveEntry, streamSupplier);
         }
     }