conga-aem-maven-plugin: Ensure last modification dates for content package ZIP file entries are kept the same when rebuilding the "all" package.

Author: stefanseifert

changes.xml

@@ -27,6 +27,9 @@
       <action type="update" dev="sseifert">
         conga-aem-maven-plugin: Check bundle status before package manager install status as the latter may report errors if certain bundles are restarted.
       </action>
+      <action type="fix" dev="sseifert">
+        conga-aem-maven-plugin: Ensure last modification dates for content package ZIP file entries are kept the same when rebuilding the "all" package.
+      </action>
     </release>
 
     <release version="2.18.2" date="2022-06-20">

tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/allpackage/AllPackageBuilder.java

@@ -557,7 +557,7 @@ private List<TemporaryContentPackageFile> processContentPackage(ContentPackageFi
                   props.put(NAME_PACKAGE_TYPE, packageType);
                 }
 
-                ZipEntry zipOutEntry = new ZipEntry(zipInEntry.getName());
+                ZipEntry zipOutEntry = newZipEntry(zipInEntry);
                 zipOut.putNextEntry(zipOutEntry);
                 props.storeToXML(zipOut, null);
                 processedEntry = true;
@@ -589,7 +589,7 @@ else if (StringUtils.equals(FilenameUtils.getExtension(zipInEntry.getName()), "z
 
               // otherwise transfer the binary data 1:1
               if (!processedEntry) {
-                ZipEntry zipOutEntry = new ZipEntry(zipInEntry.getName());
+                ZipEntry zipOutEntry = newZipEntry(zipInEntry);
                 zipOut.putNextEntry(zipOutEntry);
                 IOUtils.copy(is, zipOut);
               }
@@ -615,6 +615,17 @@ else if (StringUtils.equals(FilenameUtils.getExtension(zipInEntry.getName()), "z
     return result;
   }
 
+  private static ZipEntry newZipEntry(ZipEntry in) {
+    ZipEntry out = new ZipEntry(in.getName());
+    if (in.getCreationTime() != null) {
+      out.setCreationTime(in.getCreationTime());
+    }
+    if (in.getLastModifiedTime() != null) {
+      out.setLastModifiedTime(in.getLastModifiedTime());
+    }
+    return out;
+  }
+
   /**
    * Add dependency information to dependencies string in properties (if it does not exist already).
    * @param props Properties