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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

SpaghettiBolognese created an issue. See original summary.

SpaghettiBolognese’s picture

SpaghettiBolognese CreditAttribution: SpaghettiBolognese at Emble commented
Assigned: SpaghettiBolognese » Unassigned
Status: Active » Needs review
FileSize
file_permissions_invalid-2886765-2.patch1.34 KB
ogi’s picture

ogi CreditAttribution: 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
Primary language English
Location Vancouver
CreditAttribution: joelpittet at The University of British Columbia 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?