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

run php-fpm under workspace user #1487

Closed
wants to merge 1 commit into from

Conversation

lordofthebrain
Copy link

Run php-fpm under the workspace user. This solve all permissions problems. All files like cache files will be created with the same user

I have also add -l flag to useradd. When you use laradock with a domain user you have a large uid and your machine will be frezze by useradd without -l flag. usermod has no -l flag so I use a sed hack in php-fpm Dockerfile to avoid the frezze problem.

See: moby/moby#5419 or
https://forums.docker.com/t/run-adduser-seems-to-hang-with-large-uid/27371

I completed the 3 steps below:
  • [] I've read the Contribution Guide.
  • [] I've updated the documentation. (refer to this for how to do so).
  • [] I enjoyed my time contributing and making developer's life easier :)

@jasperf
Copy link
Contributor

jasperf commented Apr 7, 2019

Found this an interesting idea as I thought GD Library perhaps had issues resizing images due to permissions so I tried. Longer story at #1969 . Did not see this helping changing image files to being stored with permissions laradock:laradock though. And error 500 remained. So perhaps this pull request is not working this way.

@Mahmoudz Mahmoudz force-pushed the master branch 4 times, most recently from 0da1a63 to 58d1eb8 Compare July 29, 2019 14:52
@Mahmoudz Mahmoudz force-pushed the master branch 4 times, most recently from 134c78a to 493fc1a Compare August 12, 2019 19:46
@lanphan
Copy link
Contributor

lanphan commented Aug 25, 2019

already have.
Thanks for your PR.

@lanphan lanphan closed this Aug 25, 2019
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