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

ghost install should throw if any parent directory isn't readable by everyone on Linux #355

Closed
acburdine opened this issue Jul 13, 2017 · 0 comments · Fixed by #367
Closed
Assignees
Milestone

Comments

@acburdine
Copy link
Member

Summary

Because of the Linux user configuration feature of Ghost-CLI, there can now potentially be a problem if any of the parent directories of an instance aren't readable by the system. For example, the file permissions for the /root folder on Linux are 700, which means that if Ghost is installed into the /root folder, the ghost user created by the CLI won't be able to do anything as even though it has rwx permissions on the folder within /root, the parent folder doesn't allow reading, which overrides any sub-directory permissions. (this problem is the cause of #352)

The install check already does a basic "can the current user write to this directory" check, but it should be updated for linux to go back each directory to the / directory and make sure all parent directories have at least read permissions for everyone on the system.

@acburdine acburdine self-assigned this Jul 13, 2017
@acburdine acburdine added this to the 1.0 FINAL milestone Jul 13, 2017
acburdine added a commit to acburdine/Ghost-CLI that referenced this issue Jul 18, 2017
closes TryGhost#355
- add check to ensure directory tree is readable all the way to root on
linux
acburdine added a commit that referenced this issue Jul 18, 2017
closes #355
- add check to ensure directory tree is readable all the way to root on
linux
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant