On 2020/03/11 10:43, Laurence Tratt wrote: > On Thu, Mar 05, 2020 at 09:06:34PM +0000, Stuart Henderson wrote: > > Hello Stuart, > > Thanks for your comments -- I've incorporated all of them. I was waiting for > Sebastien's cargo patch to go into tree, but I'm not sure when that's > coming, and db/user.list keeps changing underneath my feet! > > > - as a daemon it would be better if this had an rc script and dedicated uid > > to run it as. probably also @sample the config file into ${SYSCONFDIR}? > > Agreed -- an rc script makes sense, at which point a dedicated uid/gid is > vital (this is a program one definitely doesn't want to run as root!). The > diff to user.list is inline. I attach an update of the port as a .tgz. I've > made a new release a) so that snare now searches /etc/snare/snare.conf b) > because one of its dependencies has been yanked. Those are relatively minor > changes, fortunately. > > > Laurie > > Index: infrastructure/db/user.list > =================================================================== > RCS file: /cvs/ports/infrastructure/db/user.list,v > retrieving revision 1.360 > diff -u -r1.360 user.list > --- infrastructure/db/user.list 9 Mar 2020 08:17:31 -0000 1.360 > +++ infrastructure/db/user.list 11 Mar 2020 10:01:35 -0000 > @@ -357,3 +357,4 @@ > 847 _iperf3 _iperf3 net/iperf3 > 848 _loki _loki sysutils/loki > 849 _synapse _synapse net/synapse > +850 _snare _snare devel/snare
Thanks - committed, we can adjust when the cargo patch goes in :)