CRM_Utils_Check_Component_Env - Soften messages for read-only extDir
authorTim Otten <totten@civicrm.org>
Wed, 28 Mar 2018 21:59:27 +0000 (14:59 -0700)
committerTim Otten <totten@civicrm.org>
Wed, 28 Mar 2018 22:10:14 +0000 (15:10 -0700)
commit37b705a0d678403b08fe89bb547917b886470837
treeb26e76c204f6ce9aa5dba91feb7852d996b22e70
parent1c6ff0a9b9251695400be4082ea08a7663b55b9c
CRM_Utils_Check_Component_Env - Soften messages for read-only extDir

There are competing schools of thought on whether extension folders should be web-writable:

* Sometimes, the most active (or the only) admins are web-based. Making the folder writeable lets them keep extensions up-to-date through the web UI.
  This includes applying security-fixes for extensions. Thus, sites with writeable extdir are harder to attack (more secure).
* Sometimes, the most active (or the only) admins don't use the web-based admin UI, and they don't trust any web-based users to do administration.
  They don't want the folder to be writeable.
* If there's a flaw that allows writing to the filesystem, it could be escalated to writing+executing code. Thus, sites with read-only
  extdir are harder to attack (more secure).

This commit tries to accept each scenario as valid -- but communicate
better.  Instead of flatly describing the read-only dir as erroreous,
present a warning with some choice/trade-off.
CRM/Utils/Check/Component/Env.php