Re: Authorization for Configuration

2018-12-07 Thread Justin Leet
You're right, I didn't really think through the specific configs that each sensor has across enrichment and indexing (because even indexing has the various batch tuning configurations). I was thinking in terms of topology, which isn't complete. Really it should (probably) be in terms of sensor all

Re: [DISCUSS] Remove `/api/v1/update/replace` endpoint

2018-12-07 Thread Nick Allen
Heads up - I have not received a response from anyone for about 3 days. I am going to take that as no one has a problem with this change. I will likely merge the PR today. On Tue, Dec 4, 2018 at 4:05 PM Nick Allen wrote: > PR #1284 completely removes the `/api/v1/update/replace` endpoint. T

Re: [DISCUSS] Remove `/api/v1/update/replace` endpoint

2018-12-07 Thread Michael Miklavcic
+1 to that Nick On Fri, Dec 7, 2018 at 11:13 AM Nick Allen wrote: > Heads up - I have not received a response from anyone for about 3 days. I > am going to take that as no one has a problem with this change. I will > likely merge the PR today. > > > > On Tue, Dec 4, 2018 at 4:05 PM Nick Allen