Fixing web user permission #5629
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
If you have another web server like tomcat on your server, the previous command end up returning tomcat instead of apache. Previously we looked at the whole line, now only at the user and the command name
BenoitLeveque
changed the title
| @@ -243,7 +243,7 @@ If there are any issues, correct them now before moving on. | |||
| $ rm -rf app/cache/* | |||
| $ rm -rf app/logs/* | |||
|
|
|||
| $ HTTPDUSER=`ps aux | grep -E '[a]pache|[h]ttpd|[_]www|[w]ww-data|[n]ginx' | grep -v root | head -1 | cut -d\ -f1` | |||
| $ HTTPDUSER=`ps axo user,comm | grep -E '[a]pache|[h]ttpd|[_]www|[w]ww-data|[n]ginx' | grep -v root | head -1 | cut -d\ -f1` | |||
There was a problem hiding this comment.
I would prefer to use --format which is imho more readable.
There was a problem hiding this comment.
i didn't find any --format options for the ps command except the o flag which allow me to specify the information I want to display
There was a problem hiding this comment.
--format isn't available on the mac version of ps
There was a problem hiding this comment.
Thanks for investigating @BenoitLeveque. I only checked on a Ubuntu machine and thought that --format was available everywhere.
|
👍 |
1 similar comment
|
+1 |
|
Nice fix! Thanks Benoît. |
wouterj
added a commit
that referenced
this pull request
Sep 30, 2015
This PR was merged into the 2.3 branch. Discussion ---------- Fixing web user permission If you have another web server like tomcat on your server, the previous command end up returning tomcat instead of apache. Previously we looked at the whole line, now only at the user and the command name Commits ------- 78ce3e9 Fix the way we found the apache/nginx user
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
If you have another web server like tomcat on your server, the previous command end up returning tomcat instead of apache.
Previously we looked at the whole line, now only at the user and the command name