|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>|
|Severity:||Affects Only Me||CC:||ecu|
Description Craig Leres 2018-03-28 22:14:23 UTC
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.
Comment 1 Nicolas Jombart 2018-04-14 18:15:22 UTC
Hi I think the best would be to go to first solution, you can consider it approved. Thanks! Nicolas
Comment 2 Craig Leres 2018-04-14 20:27:39 UTC
Created attachment 192516 [details] patch It's better for me if you'd approve the patch.
Comment 3 Nicolas Jombart 2018-04-14 20:57:04 UTC
Comment on attachment 192516 [details] patch Approved, thanks!
Comment 4 commit-hook 2018-04-14 21:16:00 UTC
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: firstname.lastname@example.org, ler (mentor) Differential Revision: https://reviews.freebsd.org/D15068 Changes: head/sysutils/sec/Makefile head/sysutils/sec/files/sec.in
Comment 5 Craig Leres 2018-04-14 21:16:43 UTC