Merge pull request #198 from rage/update_permissions_for_zipped_files

Change permissions from 744 to 755 for all zipped files

Author: Heliozoa

tmc-langs-plugins/src/tmc_zip.rs

@@ -37,14 +37,14 @@ pub fn zip_student_files<P: StudentFilePolicy>(
                 log::trace!("adding directory {}", path.display());
                 writer.add_directory(
                     path_to_zip_compatible_string(path),
-                    FileOptions::default().unix_permissions(0o744),
+                    FileOptions::default().unix_permissions(0o755),
                 )?;
             } else {
                 let bytes = file_util::read_file(entry.path())?;
                 log::trace!("writing file {}", path.display());
                 writer.start_file(
                     path_to_zip_compatible_string(path),
-                    FileOptions::default().unix_permissions(0o744),
+                    FileOptions::default().unix_permissions(0o755),
                 )?;
                 writer
                     .write_all(&bytes)

tmc-langs/src/course_refresher.rs

@@ -418,15 +418,15 @@ fn execute_zip(
             if entry.path().is_file() {
                 writer.start_file(
                     relative_path.to_string_lossy(),
-                    zip::write::FileOptions::default().unix_permissions(0o744),
+                    zip::write::FileOptions::default().unix_permissions(0o755),
                 )?;
                 let bytes = file_util::read_file(entry.path())?;
                 writer.write_all(&bytes).map_err(LangsError::ZipWrite)?;
             } else {
                 // java-langs expects directories to have their own entries
                 writer.start_file(
                     relative_path.join("").to_string_lossy(), // java-langs expects directory entries to have a trailing slash
-                    zip::write::FileOptions::default().unix_permissions(0o744),
+                    zip::write::FileOptions::default().unix_permissions(0o755),
                 )?;
             }
         }

tmc-langs/src/submission_packaging.rs

@@ -296,12 +296,12 @@ pub fn prepare_submission(
                 if entry_path.is_dir() {
                     archive.add_directory(
                         stripped.to_string_lossy(),
-                        FileOptions::default().unix_permissions(0o744),
+                        FileOptions::default().unix_permissions(0o755),
                     )?;
                 } else {
                     archive.start_file(
                         stripped.to_string_lossy(),
-                        FileOptions::default().unix_permissions(0o744),
+                        FileOptions::default().unix_permissions(0o755),
                     )?;
                     let mut file = file_util::open_file(entry_path)?;
                     std::io::copy(&mut file, &mut archive)