Re: [New] devel/snare

2020-03-11 Thread Stuart Henderson
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

2020-03-11 Thread Laurence Tratt
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

2020-03-05 Thread Stuart Henderson
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

2020-02-14 Thread Edd Barrett
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