[PATCH] Implement machinepass= in 'net ads join'
Andrew Bartlett
abartlet at samba.org
Fri Mar 1 18:11:03 MST 2013
On Fri, 2013-03-01 at 13:07 -0800, Jeremy Allison wrote:
> On Thu, Feb 28, 2013 at 11:31:17PM +1100, Andrew Bartlett wrote:
> > G'day,
> >
> > These patches add (and add tests for) a new machinepass= option for 'net
> > ads join'.
> >
> > This makes this match 'samba-tool domain join'. For master and 4.0,
> > this is not strictly required, as a workaround is to call 'samba-tool
> > domain join'. However, I've been asked to backport this to 3.6, for
> > Univention at least, and I think having the capability is a reasonable
> > addition in any case.
> >
> > The only thing is that in all these tools, we don't try and wipe the
> > password from the command line, but I'm not worried by this.
> >
> > The first patch is needed because the included tests showed that we in
> > fact did not correct set the machine password in the testsuite's calls
> > to 'samba-tool', so adding tests here has already been a worthwhile
> > exercise.
> >
> > Please review/push/comment.
>
> Having said "LGTM" and looked at the code it looks really
> obvious, but I can't push as the new test file :
>
> script/tests/test_smbclient_specified_machine_auth.sh
>
> isn't in the patchset (so my make test failed :-).
>
> Can you add it and re-submit ?
Sorry about that.
Andrew Bartlett
--
Andrew Bartlett http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-selftest-Fix-specification-of-machinepass-to-actuall.patch
Type: text/x-patch
Size: 3402 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20130302/798433c8/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-net-ads-join-Add-support-for-specifying-the-machine-.patch
Type: text/x-patch
Size: 3608 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20130302/798433c8/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-selftest-Show-that-the-machinepass-and-machinepass-j.patch
Type: text/x-patch
Size: 3416 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20130302/798433c8/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-docs-Add-documentation-for-osName-and-osVer.patch
Type: text/x-patch
Size: 1346 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20130302/798433c8/attachment-0003.bin>
More information about the samba-technical
mailing list