Skip to content
This repository has been archived by the owner on Apr 18, 2018. It is now read-only.

Drop privileges before starting other services #41

Merged
merged 1 commit into from Oct 9, 2013
Merged

Drop privileges before starting other services #41

merged 1 commit into from Oct 9, 2013

Conversation

milki
Copy link
Contributor

@milki milki commented Oct 7, 2013

pidfile is mode 666 so it's safe to drop privileges

Fixes #11

@ymilki
Drop privileges before starting other services
4cf729f 
pidfile is mode 666 so it's safe to drop privileges
@baris
Copy link
Contributor

baris commented Oct 9, 2013

ship it.

@milki milki merged commit 4cf729f into Yelp:master Oct 9, 2013
@milki
Copy link
Contributor Author

milki commented Oct 9, 2013

Reverted. root is necessary to bind to sockets...

@mrtyler mrtyler mentioned this pull request Oct 15, 2013
Open
@milki milki deleted the drop-privs-early branch October 29, 2013 22:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

application should drop privileges before starting services
3 participants
@milki @baris @ymilki