[SSSD] [PATCH] Add simple access provider

Stephen Gallagher sgallagh at redhat.com
Mon Mar 8 22:07:46 UTC 2010


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

On 03/08/2010 03:37 PM, Sumit Bose wrote:
> On Mon, Mar 08, 2010 at 11:50:38AM -0500, Stephen Gallagher wrote:
> On 03/08/2010 08:40 AM, Sumit Bose wrote:
> 
> 
>>>> This new version makes it a configuration error if both lists are
>>>> defined in the config file.
> 
>>>> bye,
>>>> Sumit
> 
> Nack (though these are nitpicks)
> 
> I'd prefer it if the simple provider had its own subdirectory like the
> LDAP, KRB5 and IPA providers.
> 
> Also, I'd rather that the interface for simple_access_check was passed
> the simple_ctx, rather than the allow_users and deny_users lists. This
> way, if we decide to add an additional feature (like resolution order),
> we're already carrying anything that the context might require.
> 
> 
>> new version attached.
> 
>> bye,
>> Sumit
> 

Ack.

- -- 
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/

iEYEARECAAYFAkuVdTIACgkQeiVVYja6o6MK7QCeNnrkA4hgvFHmE6uUT4ed8pz6
0jIAoKlouzVTnFKSrZ1C4mNRbb1k0mcW
=Ww05
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list