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

[HIGH SEVERITY] Fixes several potential directory traversal attacks #102

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@zg

zg commented Jul 28, 2015

You might want to sanitize anything that gets fed to your application through $_GET as well, but I fixed some of the more severe security issues at stake, namely this one (check the page source).

@zg zg changed the title from Fixes several potential directory traversal attacks to [HIGH SEVERITY] Fixes several potential directory traversal attacks Jul 28, 2015

@photonstorm

This comment has been minimized.

Show comment
Hide comment
@photonstorm

photonstorm Aug 4, 2015

Owner

Thanks for this, however I've decided to fix this on an Apache level.

Owner

photonstorm commented Aug 4, 2015

Thanks for this, however I've decided to fix this on an Apache level.

@photonstorm photonstorm closed this Aug 4, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment