@bx80 opened this Pull Request on March 31st 2022 Contributor

Description:

The additional access rule checks for PHP-FPM added by #18398 do not respect the enable_required_directories diagnostic config.ini.php setting and are therefore causing unwanted http requests for config files when the setting is set to disabled.

This PR adjusts this diagnostic check to only attempt file access if the config setting allows it.

Fixes #18967

Review

@bx80 commented on March 31st 2022 Contributor

I've simplified it so that the file access warning message will only be shown if they are using PHP-FPM and the enable_required_directories_diagnostic is not disabled and the file is accessible, for all other scenarios it will just show the sapi name. Also switched to using the GeneralConfig class.

This Pull Request was closed on April 4th 2022
Powered by GitHub Issue Mirror