Activity

1. svenyonson commented on Jan 12, 2022

   

   svenyonson

   on Jan 12, 2022

   Last edited by svenyonson

   Author

   More actions

   Code for above:

   `

   func onSendLogfile() {
       
       let sourceURL = Utils.LogFileUrl()
       let zipURL = Utils.ZipLogFileUrl()
       
       // Zip log file
       do {
           try FileManager().zipItem(at: sourceURL, to: zipURL);
           
           // Read entire file into a data object.
           let zipData = NSData(contentsOfFile: zipURL.path) as Data?
           if let zipData = zipData {
               mailData = ComposeMailData(subject: "This is a zipped log file",
                  recipients: ["somebody@me.com"],
                  message: "Please add additional information as needed to help us identify the issue.",
                  attachments: [AttachmentData(data: zipData, mimeType: "application/zip", fileName: "arc.logz")])
           }
       }
       catch {
           
       }
       showMailView = true
   }
   

   `
2. weichsel commented on Jan 12, 2022

   

   weichsel

   on Jan 12, 2022

   Owner

   More actions

   Hi,

   Thanks for reporting this with a detailed example.

   The ZIP format has very limited support for preserving date and time information of files. It doesn't offer a field to store time zone information so the archiving library can only assume that the stored timestamps are in the local time zone (or store that information elsewhere - e.g. in a separate file within the archive). (https://en.wikipedia.org/wiki/ZIP_(file_format)#Structure).
   Some applications and frameworks store the "creation" time zone in an extra field, but there is no idiomatic way to store that information in an archive entry.
3. svenyonson commented on Jan 12, 2022

   

   svenyonson

   on Jan 12, 2022

   Author

   More actions

   It looks like this has been addressed for other archivers?

   https://trac.edgewall.org/ticket/11162
4. weichsel commented on Jan 13, 2022

   

   weichsel

   on Jan 13, 2022

   Owner

   More actions

   Implementing the "Extended Timestamp Extra Field" would work for me. If you can provide a PR, I can review it and try to get it into an upcoming release.