[PATCH] Updated Add detailed authentication logging for NTLM authentication.

Andrew Bartlett abartlet at samba.org
Mon Mar 27 20:06:32 UTC 2017


On Thu, 2017-03-23 at 15:35 +1300, Andrew Bartlett via samba-technical
wrote:
> On Wed, 2017-03-22 at 18:48 -0700, Jeremy Allison wrote:
> > On Thu, Mar 23, 2017 at 11:45:49AM +1300, Andrew Bartlett via
> > samba-
> > technical wrote:
> > > 
> > > We think we are finally ready to have this branch pushed to
> > > master.
> > >  
> > > 
> > > I'm currently testing if we can disconnect the tests from the
> > > messaging
> > > bugs I found while we debate API preferences.
> > > 
> > > If we can split it, or otherwise get past this, we hope to push
> > > as
> > > much
> > > possible soon, so if you or anybody else would like to review it
> > > that
> > > would be most helpful!
> > > 
> > > git://git.catalyst.net.nz/samba.git auth-logging
> > > 
> > > http://git.catalyst.net.nz/gitweb?p=samba.git;a=shortlog;h=refs/h
> > > ea
> > > ds/a
> > > uth-logging
> > > 
> > > We didn't ask for formal review/push yet because we wanted to
> > > address
> > > the concern you raised early, that we might not work between some
> > > of
> > > the early commits.  We have written an extensive test suite, and
> > > run
> > > that between all the commits that change APIs via:
> > > 
> > > git rebase -i
> > > x make test TESTS=auth_log
> > > 
> > > Therefore we are now confident we do not have any of the feared
> > > issues
> > > there. 
> > > 
> > > This is a massive improvement in Samba's suitability for as an AD
> > > DC,
> > > and I'm very proud of the work done here by Gary in particular. 
> > 
> > Sounds good ! Can you post it as a patchset to the list as
> > well, or is it too big ?
> 
> It is huge (500k) partly because it builds on metze's branch, thanks
> to
> his rebase-for-hire services ;-)

The auth-logging patch is still 500k, but I attach it here for the
record.  This is without the patches it depends on, so can't be pushed
on it's own, but should allow some review.

We are sorting out some last issues, but it is almost ready.

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: auth-logging.patch
Type: text/x-patch
Size: 510840 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20170328/7280b075/auth-logging-0001.bin>


More information about the samba-technical mailing list