Ensure correct permissions for decompressed files from .zip on Unix

src/cli.rs

@@ -127,7 +127,7 @@ impl TryFrom<clap::ArgMatches<'static>> for Command {
                         }
 
                         eprintln!("{} {}", "[ERROR]".red(), err);
-                        return Err(crate::Error::IOError);
+                        return Err(crate::Error::IoError);
                     }
                 }
 

src/decompressors/zip.rs

@@ -10,6 +10,15 @@ use zip::{self, read::ZipFile, ZipArchive};
 use super::decompressor::{DecompressionResult, Decompressor};
 use crate::{file::File, utils};
 
+#[cfg(unix)]
+fn __unix_set_permissions(file_path: &PathBuf, file: &ZipFile) {
+    use std::os::unix::fs::PermissionsExt;
+
+    if let Some(mode) = file.unix_mode() {
+        fs::set_permissions(&file_path, fs::Permissions::from_mode(mode)).unwrap();
+    }
+}
+
 pub struct ZipDecompressor {}
 
 impl ZipDecompressor {
@@ -65,7 +74,8 @@ impl ZipDecompressor {
                 io::copy(&mut file, &mut outfile)?;
             }
 
-            // TODO: check if permissions are correct when on Unix
+            #[cfg(unix)]
+            __unix_set_permissions(&file_path, &file);
 
             let file_path = fs::canonicalize(file_path.clone())?;
             unpacked_files.push(file_path);
@@ -89,6 +99,7 @@ impl ZipDecompressor {
             None => {
                 let file = fs::File::open(&from.path)?;
                 let mut archive = zip::ZipArchive::new(file)?;
+
                 Ok(Self::zip_decompress(&mut archive, into)?)
             }
         }

src/error.rs

@@ -9,7 +9,7 @@ pub enum Error {
     // TODO: get rid of this error variant
     InvalidUnicode,
     InvalidInput,
-    IOError,
+    IoError,
     FileNotFound(PathBuf),
     AlreadyExists,
     InvalidZipArchive(&'static str),
@@ -50,7 +50,7 @@ impl From<std::io::Error> for Error {
             std::io::ErrorKind::AlreadyExists => Self::AlreadyExists,
             _other => {
                 println!("{}: {}", "IO error".red(), err);
-                Self::IOError
+                Self::IoError
             }
         }
     }