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.

Sign up for GitHub

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

Jump to bottom

Permission issues introduced when running artisan tasks as root #6

Open
petehalverson opened this issue Dec 5, 2017 · 2 comments
Open

Permission issues introduced when running artisan tasks as root #6

petehalverson opened this issue Dec 5, 2017 · 2 comments

Comments

@petehalverson
Copy link
Member

Permissions on files changed by running command line tasks has the potential to reset permissions as root:root (e.g. file based cache)
@petehalverson petehalverson changed the title Potential permissions issue Permission issues introduced when running artisan tasks as root Dec 6, 2017
@petehalverson
Copy link
Member Author

Can be resolved by passing the user flag on run or exec:

  • docker run -it --user www-data aspendigital/octobercms bash
  • docker exec --user www-data octobercms bash php artisan plugin:install aspendigital.resizer
  • docker-compose exec --www-data octobercms bash

petehalverson added a commit that referenced this issue Dec 7, 2017
@petehalverson
Permissions update
f640d17 
Set permissions + setfacl on `./plugins`, `./storage`, `./themes`

Fixes Issue #6
petehalverson added a commit that referenced this issue Dec 7, 2017
@petehalverson
Move permissions + setfacl to entrypoint
15b5c01 
`docker build` doesn’t reliably support files ACLs

Issue #6
petehalverson added a commit that referenced this issue Dec 8, 2017
@petehalverson
Permissions fix
32e5ae8 
Move chmod g+ws to docker build; fix script to include subdirectories - Issue #6
@petehalverson petehalverson reopened this Dec 8, 2017
@petehalverson
Copy link
Member Author

petehalverson commented Dec 8, 2017 •
edited
Loading

This is still an issue. setfacl fails when bind mounting volumes (3da3801)

petehalverson added a commit that referenced this issue Jun 4, 2018
@petehalverson
Set CMS defaultMask to allow group access
a362320 
Issue #6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@petehalverson