Page MenuHomePhabricator

Need to disable phpinfo...
Closed, WontfixPublic

Description

Phabricator pre-flight checks currently prevent startup if the following is defined in the php config:

disable_functions = phpinfo

We are required to disable phpinfo as standard on all php installations to improve base security.

Could I request phabricator allows disabling of phpinfo within the php application - as currently we need to manually hack the source code on each phabricator update to re-apply this exception.

Event Timeline

epriestley claimed this task.
  • The "root problem" from our point of view is a security policy with no technical basis. As a general rule, we do not implement changes to help installs solve problems of their own making, and you have this problem only because of your policy.
  • disable_functions = phpinfo does not improve the security of Phabricator, but impairs normal behavior.
  • To move forward, you should either: change the policy, obtain an exemption to the policy, fork Phabricator, or select other software which adheres to the policy.

Cool thanks for the detailed response.

I do think it is technically reasonable to have phpinfo disabled by default, as in normal running mode there is no need for it and it does open doors that would be better closed. This is common practice in a few places so thought i'd share upstream with you guys.

random write up that explains in more words - https://www.adampalmer.me/iodigitalsec/2010/07/06/disabling-phpinfo/

Though obviously i accept its your call :)