eaccelerator/eaccelerator — pull request #17 by remicollet (2012-09-09 11:39:24+00:00)
Title: Try to improve cache management.

* make cache_dir and log_file option "PERDIR'"

* drop the chmod(777)

777 right seems really ugly and will be consider by a lot of people as a security issue.

Proposal.

* With default config (/tmp/eaccelerator) when we create the dir, keep the 777 (or 1777), this seems acceptable under /tmp (already 1777), expecially on recent system running systemd with "privatetmp".

* Is admin, change this value, it will have to create the dir with correct access.

For example, for RPM packaging, I will use, for apache

/etc/httpd/conf.d/eaccelerator.conf => using php_value for cache_dir and log_file. This will be apache only.

/var/cache/php-eaccelerator => own by root (755)

/var/cache/php-eaccelerator/48 => own by apache (700)

Tested here, seems ok,
Please review.