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

application should drop privileges before starting services #11

Open
baris opened this issue Jun 11, 2013 · 2 comments · Fixed by #41
Open

application should drop privileges before starting services #11

baris opened this issue Jun 11, 2013 · 2 comments · Fixed by #41

Comments

@baris
Copy link
Contributor

baris commented Jun 11, 2013

This is currently what application does and causes background services to start as root.

                self.start_services()
                pid.write(self.pid_file, append=True)

                # Drop privileges
                uid = pwd.getpwnam(Settings.get("username", "www-data"))[2]
                os.setuid(uid)

                tornado.ioloop.IOLoop.instance().start()
@mrtyler
Copy link
Contributor

mrtyler commented Oct 15, 2013

Looks like you reverted pr #41. Should this be re-opened?

@milki
Copy link
Contributor

milki commented Oct 15, 2013

Yes, it should be reopened. I didn't see a button to do that.

On Tue, Oct 15, 2013 at 10:16 AM, Tyler Roscoe notifications@github.comwrote:

Looks like you reverted pr #41#41.
Should this be re-opened?


Reply to this email directly or view it on GitHubhttps://github.com//issues/11#issuecomment-26353876
.

milki

@mrtyler mrtyler reopened this Oct 15, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants