htmlpurifier sets different permissions in dataroot

Bug #1333071 reported by Simon Coggins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Low
Simon Coggins

Bug Description

Currently HTML purifier uses its own default directory and file permissions when creating files in dataroot, instead of respecting those given by $CFG->directorypermissions. That can mean some files generated have stricter permissions than specified.

To reproduce:

1. Create a config.php file with $cfg->directorypermissions = 0777
2. Install a mahara site via the command line CLI script
3. Visit the homepage of the site in a browser
4. View the dataroot folder and look at the file permissions

You will notice that most folders have 777 permissions but the subdirectories inside htmlpurifier/ have 755.

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/3457

Changed in mahara:
importance: Undecided → Low
status: New → In Progress
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/3457
Committed: http://gitorious.org/mahara/mahara/commit/41d4424759416fc34a33e2336a84d45fd88f930f
Submitter: Son Nguyen (<email address hidden>)
Branch: master

commit 41d4424759416fc34a33e2336a84d45fd88f930f
Author: Simon Coggins <email address hidden>
Date: Fri Jun 20 20:14:16 2014 +1200

Configure htmlpurifier to use same directory permissions as Mahara (Bug #1333071)

This is needed to ensure the permissions of files/folders in dataroot
are correct for unit testing

Change-Id: I7877c22e0a5140e07d2c4fddaeb10e2f5ffc34ad
Signed-off-by: Simon Coggins <email address hidden>

Changed in mahara:
status: In Progress → Fix Committed
milestone: none → 1.10.0
assignee: nobody → Simon Coggins (simon-coggins)
Aaron Wells (u-aaronw)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.