[SSSD] [PATCH] Add simple access provider

Stephen Gallagher sgallagh at redhat.com
Mon Mar 8 16:50:38 UTC 2010


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

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.

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

iEYEARECAAYFAkuVKt4ACgkQeiVVYja6o6MG9gCfdoX+aOs9QghSnGvoIhJhA4bp
KJQAoJPWFEevho5mxv3lJ5LPLWKAg4vD
=A0CV
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list