Re: [New] devel/snare
On 2020/03/11 10:43, Laurence Tratt wrote: > On Thu, Mar 05, 2020 at 09:06:34PM +, 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 - 1.360 > +++ infrastructure/db/user.list 11 Mar 2020 10:01:35 - > @@ -357,3 +357,4 @@ > 847 _iperf3 _iperf3 net/iperf3 > 848 _loki_loki sysutils/loki > 849 _synapse _synapsenet/synapse > +850 _snare _snare devel/snare Thanks - committed, we can adjust when the cargo patch goes in :)
Re: [New] devel/snare
On Thu, Mar 05, 2020 at 09:06:34PM +, 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 - 1.360 +++ infrastructure/db/user.list 11 Mar 2020 10:01:35 - @@ -357,3 +357,4 @@ 847 _iperf3_iperf3 net/iperf3 848 _loki _loki sysutils/loki 849 _synapse _synapsenet/synapse +850 _snare _snare devel/snare snare_port.tgz Description: application/tar-gz
Re: [New] devel/snare
On 2020/03/02 21:21, Laurence Tratt wrote: > This is a new port for snare, a GitHub webhooks runner daemon. From > pkg/DESCR: > > snare is a GitHub webhooks runner daemon. When snare receives a webhook > event from a given repository, it authenticates the request, and then > executes a user-defined "per-repo program" with information about the > webhook event. > > More info at https://tratt.net/laurie/src/snare/ > > This version of the port is for snare 0.2.0; edd@ OK'd the 0.1.0 version of > this port. - "MODCARGO_CRATES_UPDATE += libc" isn't doing anything because the version of libc in the MODCARGO_CRATES list is the same as the version in Cargo.lock so please remove that - if semarie's cargo diff goes in before this is committed, the openssl-sys patch should be removed - 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}? - I'd prefer if the wasi MODCARGO_CRATES line was left how it came from modcargo-gen-crates-licenses rather than splitting the license onto the line before, i.e. -# Apache-2.0 WITH LLVM-exception OR Apache-2.0 OR MIT -MODCARGO_CRATES += wasi0.9.0+wasi-snapshot-preview1 +MODCARGO_CRATES += wasi0.9.0+wasi-snapshot-preview1# Apache-2.0 WITH LLVM-exception OR Apache-2.0 OR MIT
Re: [New] devel/snare
On Thu, Feb 13, 2020 at 02:19:28PM +, Laurence Tratt wrote: > snare is a GitHub webhooks daemon. I reviewed this prior to Laurie posting it on-list. I'd be happy to commit this if someone OKs. -- Best Regards Edd Barrett http://www.theunixzoo.co.uk