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

Sandboxed webroot #100

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

Conversation

adamnfish
Copy link

This creates a www webroot directory that can be used as the root for webserver configuration. The benefit to this approach is that files in the main directory are no longer exposed on the website. This includes the data directory, the PHP source code (beyond the required index.php entry point) and any other files that get placed in the root of the project during deployment or while running.

This creates a www webroot directory that can be used as the root for
webserver configuration. The benefit to this approach is that files in
the main directory are no longer exposed on the website. This includes
the data directory, the PHP source code (beyond the required index.php
entry point) and any other files that get placed in the root of the
project during deployment or while running.
@ulikoehler
Copy link

I agree, this pull request should be merged, it is quite useful indeed. Note, however, that this repository is pretty much not maintained by the author and AFAIK no-one has commit rights there.

@TheFiZi
Copy link

TheFiZi commented Nov 27, 2015

You might try submitting to this fork: https://github.com/elrido/ZeroBin

Development has been fairly active over there.

@elrido
Copy link

elrido commented Nov 29, 2015

Thanks for mentioning our fork. We do already support this mode of installation and have documented this in the installation guide. I.e. snip.dssr.ch is set up like this.

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.

4 participants