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

Andrew Bartlett abartlet at samba.org
Fri Jul 6 19:45:44 UTC 2018


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!

Andrew Bartlett

-- 
Andrew Bartlett                       http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba




More information about the samba-technical mailing list