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

No more permissions craziness! #8795

Merged
merged 3 commits into from
Dec 1, 2017
Merged

No more permissions craziness! #8795

merged 3 commits into from
Dec 1, 2017

Conversation

weaverryan
Copy link
Member

No description provided.

@weaverryan weaverryan added this to the 4.0 milestone Nov 30, 2017
Nyholm
Nyholm reviewed Nov 30, 2017
View reviewed changes
setup/file_permissions.rst Outdated
.. _`enable ACL support`: https://help.ubuntu.com/community/FilePermissionsACLs
If you decide to store log files on disk, you *will* need to make sure your
logs directory (e.g. ``var/logs``) is writable by your web server user and
terminal user.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May be add how would I do that?

@weaverryan
Copy link
Member Author

Added a note about 777'ing the logs if you need to. There are other ways to do this... but I don't think we can really go into a rabbit hole here with that :)

@weaverryan
Copy link
Member Author

Thanks to the people who actually made this possible. I just got to do the fun part (delete the page)

@weaverryan weaverryan merged commit 32a2516 into symfony:4.0 Dec 1, 2017
weaverryan added a commit that referenced this pull request Dec 1, 2017
@weaverryan
feature #8795 No more permissions craziness! (weaverryan, javiereguiluz)
5489235 
This PR was merged into the 4.0 branch.

Discussion
----------

No more permissions craziness!

Commits
-------

32a2516 Minor fixes
327db54 adding details about 777
d50d0e6 Bye bye permissions craziness
@weaverryan weaverryan deleted the no-more-permissions branch December 1, 2017 18:47
@ostrolucky
Copy link
Contributor

ostrolucky commented Mar 6, 2020 •
edited
Loading

I am interested for reasoning why was this done. Can you post some references or/and explanation? I mean zealous chmod 777.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Nyholm Nyholm Nyholm left review comments

@javiereguiluz javiereguiluz javiereguiluz approved these changes

Assignees
No one assigned
Labels
Status: Reviewed
Projects
None yet
Milestone
4.0
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@weaverryan @ostrolucky @javiereguiluz @Nyholm @carsonbot