[IO-575] copyDirectory (all overloads) does not maintain file permissions

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.5
• Fix Version/s: 2.9.0, 2.12.0

Environment

Merged https://github.com/apache/commons-io/pull/377, see the PR for behavior.

COPY_ATTRIBUTES is no longer used by default.

• Assignee: Unassigned
• Reporter: Adam Kewley
• Created: 18/Mar/18 12:33
• Updated: 18/Apr/23 21:52
• Resolved: 06/Sep/22 16:10

Description

I found that permissions (specifically, execute) are not maintained when using copyDirectory. The following test demonstrates the behavior:**

 

Path sourceDir = Files.createTempDirectory("source-dir");
String filename = "some-file";
Path sourceFile = Files.createFile(sourceDir.resolve(filename));

assertThat(sourceFile.toFile().canExecute()).isFalse();

sourceFile.toFile().setExecutable(true);

assertThat(sourceFile.toFile().canExecute()).isTrue();

Path destDir = Files.createTempDirectory("some-empty-destination");

FileUtils.copyDirectory(sourceDir.toFile(), destDir.toFile());

Path destFile = destDir.resolve(filename);

assertThat(destFile.toFile().exists()).isTrue();
assertThat(destFile.toFile().canExecute()).isTrue(); // fails

Is it working as intended that the permissions are not copied over?