search

Iolaum / fcust

Linux Common Folder Custodian
Other
0 stars 0 forks source link

Incompatible log folder permissions - Fixed, Add tests #3

Closed Iolaum closed 3 years ago

Iolaum commented 3 years ago

Description

Running fcust from the second user results in a permission error because they don't have access to enter the /tmp/fcust folder. This is becuase by default the folder has 775 permissions.

One solution is to have the user creating the folder to change it's group to the common group used by the users of the common folder.

Note: This may not scale if there are more than 1 common folders or user combinations. (Not worrying about that for now though.)

Iolaum commented 3 years ago

3cb9b325c76d6cf26f1c60b8e670d87f4357b20e hotfixes the issue but it'd also be good to add some tests around this.

Iolaum commented 3 years ago

c7bed120b2eba2631168caf9401030f2da5cb2cd added test for logging as well