[SSSD] [PATCH] Check cache_credentials in sysdb_cache_auth_send()

Stephen Gallagher sgallagh at redhat.com
Tue Feb 2 19:44:16 UTC 2010


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/01/2010 01:24 PM, Stephen Gallagher wrote:
> On 01/26/2010 11:43 AM, Sumit Bose wrote:
>> I'm not sure this is a good idea, unless you want to force
>> provider=local domains to have cache_credentials=true. Right now, this
>> will break authentication against the LOCAL domain if cache_credentials
>> is not set.
> 
> 
>>> Currently provider=local domains do not use sysdb_cache_auth_send()
>>> although it might be a good idea let them use it to have only one place
>>> where the password hashes are compared.
> 
>>> To make this work we should check for (cache_credentials==true ||
>>> strcmp(domain->name, "local") == 0) and add a new option similar to
>>> offline_credentials_expiration for local domains.
> 
>>> But I would prefer to do this in a separate patch.
> 
>>> What do you think?
> 
> I think this is fine, then.
> 
> Ack to this patch, but please consolidate the hashed password checks in
> the future.
> 

Pushed to master.

- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAktogI8ACgkQeiVVYja6o6PXeQCffeGaFajnoISQiubMqARiWujY
DSsAoK7oWNAM5SbqAKe1qEyWnNkevPtK
=LfqL
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list