[Patch] Make samba4 return a correct Supported Encryption

Andrew Bartlett abartlet at samba.org
Fri Sep 11 03:48:14 MDT 2009


On Fri, 2009-09-11 at 09:07 +0000, Matthias Dieter Wallnöfer wrote:
> Matthieu,
>  
> to be honest, I find my reworked version now very clear. We ask for
> the attributes where we need them. Sure, we'd have some small speedup
> (one lookup less) but if you consider the LDB modules  - also they
> often have more lookups than really needed. More lookups generally
> means to be more clear in design.
>  
> The other point is (like Andrew mentioned) that the design of this
> call still isn't completed fully yet. There is missing the
> "dNSDomainName" and "servicePrincipalName" handling (maybe some more
> checks, also the "samldb" module will be enhanced).
>  
> So if you really want this patch in - please repost it after the final
> design.
> 

I would still prefer one search.  It means one set of error handling,
and clearer behaviour.

I don't see how pre-declaring the attributes we need, then asking for
them at the right moment is really that hard.

Andrew Bartlett

-- 
Andrew Bartlett                                http://samba.org/~abartlet/
Authentication Developer, Samba Team           http://samba.org
Samba Developer, Cisco Inc.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20090911/4646ed0b/attachment.pgp>


More information about the samba-technical mailing list