[RFC PATCH 0/2] add JSON output to net ads

[Jeremy Allison]

On Sat, Jul 07, 2018 at 07:45:44AM +1200, Andrew Bartlett wrote:
> On Fri, 2018-07-06 at 10:37 -0700, Jeremy Allison via samba-technical
> wrote:
> > On Fri, Jul 06, 2018 at 04:48:51PM +0200, Philipp Gesang via samba-technical wrote:
> > > Hey guys,
> > > 
> > > the informational commands of the ``net ads'' family format their output in an
> > > ad-hoc manner that is not specified. For automated processing it would be
> > > useful to optionally encode the output as JSON.
> > > 
> > > These two patches against master add such a JSON output to ``net ads info'' and
> > > ``net ads lookup''. They have been forward-ported from the 4.7-ish Samba that
> > > we run. Compilation succeeds with master but I'm still in the process of getting
> > > working test environment, so they're virtually untested.
> > > 
> > > I'd appreciate your feedback.
> > 
> > Sorry, I'm going to have to do a NAK on this work
> > until the json_XXX() code is fixed to stop returning
> > (void) on errors.
> > 
> > The longer this sore remains in the code, the more
> > code we'll have to fix until it's done correctly.
> > 
> > I'm going to NAK anything adding more (void)json_XXX()
> > calls until this gets fixed.
> 
> The patches to get that done are here:
> 
> https://gitlab.com/catalyst-samba/samba/commits/gary-audit-lib
> 
> I had to pull Gary onto a more urgent task which is why this didn't get
> submitted yet.
> 
> Philipp,
> 
> If you base your work on that (and add the tests you already mention
> are missing) then hopefully we can get this into master in time.
> 
> Thanks for your interest in JSON formatting!

Oh thanks so much for that ! I'm 100% happy to review
once you think it's in a mergeable state :-).

Jeremy.