Add robustness to file permission errors in search

halfer / awooga-app

Web application to report coding tutorials of non-optimal quality

0 stars 0 forks source link

Add robustness to file permission errors in search #13

Closed halfer closed 9 years ago

halfer commented 9 years ago

The search index shares the mounted file system with the repository fetcher. These need different permissions/owners, and as such can easily stop working if a permissions "fix" is made manually on the server.

A good start would be to do a read test prior to a search and a write test prior to a web-based save. If either of these fail, a graceful error can be displayed. Bonus points for marking records as needing indexing after a write of any kind, with an indexer operating on a cron. This will make creating/updating records less brittle.

halfer commented 9 years ago

It would be good also to add a script to correctly fix permissions if they come out of sync, and for initial setup (nothing in the INSTALL file at present). We need:

filesystem/mount/search-index: group owner = www-data, 660
filesystem/mount/everything-else (i.e. repos): group owner = awooga, 660

The repos could do with their own subfolder too.

halfer commented 9 years ago

I've moved the repo pulls into filesystem/mount/repos so they can have their own perms, and added more perm resets into the mount script. Seems to work!