Usability of 'samba-tool domain provision'
repenny241155 at gmail.com
Thu Jun 2 19:44:31 UTC 2016
On 02/06/16 20:21, Andrew Bartlett wrote:
> On Thu, 2016-06-02 at 15:19 +0100, Rowland Penny wrote:
>> On 02/06/16 14:02, Andrew Bartlett wrote:
>>> On Thu, 2016-06-02 at 12:28 +0100, Rowland Penny wrote:
>>>> Ok, before I alter my patches, can you confirm, it is just the
>>>> 'realm' & 'domain' from options to args, you are against, or is
>>>> anything else ?
>>> The user should not be prompted about use_xattrs. In fact, just to
>>> demonstrate that Samba development is always an art both of knowing
>>> rules and of knowing when not to apply them, I would argue we
>>> actually drop the --use-xattrs option, or at least hide it.
>> If I understand you correctly, don't ask about 'use_xattrs', remove
>> option and set 'use_xattrs' to 'auto'.
> Yes, remove it from interactive, setting to auto.
> Then add "(deprecated)" to the help text for --use-xattrs, so we can
> get rid of it in another release cycle (our standard is to give one
> release of notice for these things).
OK, will do.
>>> For the password check, I would prefer if we re-used the same
>>> complexity check as the DB will use, perhaps by adding a python
>>> to the C function in use. Otherwise I fear the two will diverge,
>>> just cause further confusion.
>> Ah, here again, this is something I have never done, which 'C'
>> are you referring to ? and is there anything in samba-tool that
>> does something similar, that I can examine for clues :-)
> Sadly no, and creating new C bindings isn't trivial either. I realise
> this is harder, but I would still prefer it was done right.
I did some investigation into this and I think it is going take me some
time to get my head around it, so I will remove the password patch until
such time as I get can get it to work :-)
I will post a new set of patches as soon as possible.
> The C fuction is samdb_check_password() in source4/dsdb/common/util.c
> and the bindings for this kind of thing are via source4/dsdb/pydsdb.c
> via python/samba/samdb.py.
> Andrew Bartlett
More information about the samba-technical