This module ignores the file_chmod_directory and file_chmod_file variables. In our setup this will cause an error with the sticky-bit. I think this module should use drupal_chmod() and drupal_mkdir() instead of the normal mkdir/chmod methods. Patch will be in the comments.

CommentFileSizeAuthor
#2 file_permissions_invalid-2886765-2.patch1.34 KBspaghettibolognese

Comments

SpaghettiBolognese created an issue. See original summary.

spaghettibolognese’s picture

spaghettibolognese commented
Assigned: spaghettibolognese » Unassigned
Status: Active » Needs review
StatusFileSize
new file_permissions_invalid-2886765-2.patch1.34 KB
ogi’s picture

ogi commented

Could you be more specific what breaks with current 0700/0600 modes?

The problem is that File Cache doesn't just manage files (e.g. uploaded ones) but also may contain security information from the DB.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

@SpaghettiBolognese I'd like to know what's wrong with the modes, the patch looks like a good idea to follow drupal's wrappers but why the mode change?

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Status: Needs review » Closed (outdated)

Drupal 7 is end-of-life, closing.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.