Bug 227056
| Summary: | sysutils/sec: Can we allow sec_user to update pidfile? | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Craig Leres <leres> | ||||
| Component: | Individual Port(s) | Assignee: | Craig Leres <leres> | ||||
| Status: | Closed FIXED | ||||||
| Severity: | Affects Only Me | CC: | ecu | ||||
| Priority: | --- | Flags: | ecu:
maintainer-feedback+
|
||||
| Version: | Latest | ||||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| Attachments: |
|
||||||
Hi I think the best would be to go to first solution, you can consider it approved. Thanks! Nicolas Created attachment 192516 [details]
patch
It's better for me if you'd approve the patch.
Comment on attachment 192516 [details]
patch
Approved, thanks!
A commit references this bug: Author: leres Date: Sat Apr 14 21:15:24 UTC 2018 New revision: 467341 URL: https://svnweb.freebsd.org/changeset/ports/467341 Log: Move the pidfile from /var/run/sec.pid to /var/run/sec/sec.pid and chown /var/run/sec when sec_user is set. PR: 227056 Reviewed by: ler (mentor) Approved by: ecu@200ok.org, ler (mentor) Differential Revision: https://reviews.freebsd.org/D15068 Changes: head/sysutils/sec/Makefile head/sysutils/sec/files/sec.in Committed, thanks! |
We run sec as a non-root user. However the rc.d script does not provide a way to change the pid file. I can think of several ways to solve this: - Change pidfile to /var/run/sec/sec.pid Optionally chown /var/run/sec to sec_user - Optionally add sec_pidfile to rc.conf to override pidfile If the maintainer could provide feedback on the preferred solution I'm happy to create a patch for approval.