Skip to content
This repository has been archived by the owner on Nov 25, 2020. It is now read-only.

security with additional repositories in data folder #635

Closed
FotoVI opened this issue Sep 3, 2014 · 4 comments
Closed

security with additional repositories in data folder #635

FotoVI opened this issue Sep 3, 2014 · 4 comments

Comments

@FotoVI
Copy link

FotoVI commented Sep 3, 2014

Although it is recommended to create additional repositories outside of the web root,
many users create them inside of the data folder but lack knowledge about editing the .htaccess rules.
Thus, even repositories intended for private use are actually public.

I suggest setting access rules for the data folder in general to:

<../data/.htaccess>
    Order Deny,Allow
    Deny from all

And allowing access for public repositories:

<../data/publicfolder/.htaccess>
    Order Deny,Allow
    Allow from all
    <Files ".ajxp_*">
        Deny from all
    </Files>

The public access rule creation can be implemented as a feature that is set during repository creation, so even users without knowledge about .htaccess files get to create public repositories,
while enjoying security for private ones.

@cdujeu
Copy link
Member

cdujeu commented Sep 8, 2014

and that way we could also remove the .htaccess files under data/cache, data/tmp, etc... (they would be deny for all by default, right)?

@FotoVI
Copy link
Author

FotoVI commented Sep 9, 2014

yes, those are redundant in that case

@FotoVI
Copy link
Author

FotoVI commented Sep 9, 2014

You can start implementing this configuration right away,
the only variable I can see are newly created public folders by the user,
which need a .htaccess file created on the fly or a messagebox for admin provisionally.

@cdujeu
Copy link
Member

cdujeu commented Sep 18, 2014

impl. in dev branch

@cdujeu cdujeu closed this as completed Sep 18, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants