Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unexpose php version #48

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

unexpose php version #48

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented May 28, 2019

Request to disable php_expose. Because when enabled it discloses information about the system that could indicate vulnerabilities or help malicious individuals otherwise.

php_expose is a core INI directive, hence it can only be configured in the primary php.ini.

The main reason for myself is that several penetration testing tools indicate it as a (low) security risk. E.g.:

Tradeoff: I am proud to be hosting my app at Azure which means I can leave the security patching to the people who like it, are good at it and make it their responsibility. Bu now there is no longer a header on the HTTP responses showing off my app's high PHP version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants