-
Notifications
You must be signed in to change notification settings - Fork 133
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
composer create-project, the permissions of all directories is 777? #30
Comments
I can only confirm that some directories have permissions set to 777, not all as you indicate:
|
I'm not entirely sure (yet) why this happens. If I run |
this image (https://hub.docker.com/r/composer/composer/) works well on permissions, |
I had a look at the |
It seems related to the |
@alcohol did you've a chance to take a look? I had to revert to previous composer container because of this bug. |
No I have not yet had a chance. Among other things, I still need to check if this other image still behaves correctly when I introduce the exact same Composer version into it. Right now I think it is a bit outdated in comparison. They also use a different base image, but debugging if that makes a difference will take a bit more work :-/ |
Odd, I've gone over the wordplate install in several ways. -- host git clone
dockerized git clone
host untar github tarball
dockerized untar github tarball
host unzip github zipfile
dockerized unzip github zipfile
-- So it would seem this happens specifically when composer uses the zipfile archives from github (though tarballs also show a peculiar difference). But only inside our docker container. At least I have something I can debug further now 👍 |
Hi, anything new on this bug? |
No, unfortunately I do not have a lot of free time to spend on open-source. This issue is still on my to-do list, but unfortunately, so are quite a lot of other issues. |
I see. I will try to poke around, maybe I can help. |
I actually had a hunch right now and did a quick check, and might have found a solution specifically for the zip issue. It seems the default |
nice! thanks :) |
Still have to wait for this to be merged, and even then I am not sure if that will update the current tags already available on Docker Hub. Might have to wait for the next release. |
as i use this docker image, it works well with
composer install
andcomposer update
, but i create a composer project, like:i find that the permissions of all directories except
vendor
is 777. even command withps,
uname -a
umask
The text was updated successfully, but these errors were encountered: