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

Consider blocking the Proxy header in default ht.access to protect against httpoxy #13076

Closed
Mark-H opened this issue Jul 19, 2016 · 2 comments
Closed

Comments

@Mark-H
Copy link
Collaborator

Mark-H commented Jul 19, 2016

Summary

The httpoxy vulnerability (https://httpoxy.org/) might have an impact on MODX or extras, depending on server configuration. This can not be mitigated from PHP, but for apache a htaccess blob can help protect against it so we could include that in the default ht.access.

Observed behavior

Certain aspects of MODX or third party components might be vulnerable. While I don't think MODX or the extras would be to blame, we can help guide people a little bit by adding mitigation into the standard ht.access file in the download.

Expected behavior

The following snippet could be included in the distributed ht.access, which would then automatically be applied when people set up friendly urls with it:

<IfModule mod_headers.c>
   RequestHeader unset Proxy
</IfModule>

It'd probably need to have some comments explaining why it's necessary

Environment

The snippet above is specific to Apache, taken from the official recommendations at https://httpoxy.org/#fix-now which also has snippets for nginx and others.

@amdbuilder
Copy link

I would argue this issue should be handled by your host. Adding this seems like a slippery slope, at what point do you stop adding security fixes to the .htaccess and make the host accountable?

I know cPanel added the fix, so any cPanel hosts that update their systems will be protected. If they don't update I would consider looking elsewhere.

@Mark-H
Copy link
Collaborator Author

Mark-H commented Aug 14, 2017

Agreed.

@Mark-H Mark-H closed this as completed Aug 14, 2017
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

No branches or pull requests

2 participants