[SSSD] [PATCHES] Two patches related to SSSDDomain.remove_provider()

Jakub Hrozek jhrozek at redhat.com
Wed Dec 9 03:48:29 UTC 2009


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

On 12/07/2009 11:04 PM, Stephen Gallagher wrote:
> Patch 0001: remove_provider() really doesn't need the provider name
> (ldap, krb5, etc.) specified in order to remove it. This patch
> eliminates that argument to the function.
> 

ACK

Good change to the API.

> Patch 0002: remove_provider() needs to clean up any options that were
> set for that provider. This patch does so, as well as ensuring that it
> does not clean up options that may be needed by other providers (such as
> krb5_realm, which may be used by the ldap provider)
> 

ACK but please fix whitespace errors before pushing.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAksfHgcACgkQHsardTLnvCVIXwCgqWg3qRBAMEG7LsFtoxfT6D7w
JZIAnA3jRNAQyGO9zt+tcUggBsVgJ5J7
=3w1b
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list