[SSSD] [PATCH] Convert krb5_kdcip option to krb5_server
Stephen Gallagher
sgallagh at redhat.com
Wed Sep 29 11:36:27 UTC 2010
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 09/29/2010 07:24 AM, Jan Zelený wrote:
> Jan Zelený <jzeleny at redhat.com> wrote:
>> Jan Zelený <jzeleny at redhat.com> wrote:
>>> Based on Jakub's proof-of-concept patch I prepared a patch which handles
>>> this option replacement. As it was discussed on the meeting couple weeks
>>> back, I didn't modify the update script (I hope I remember this
>>> requirement modification right) and in case both krb5_kdcip and
>>> krb5_server options are in the config file, the latter one has bigger
>>> priority.
>>>
>>> Jan
>>
>> Sorry, wrong patch. Will send the most recent version later today.
>
> Here it is.
>
> Jan
Nack.
You can't remove krb5_kdcip from SSSDConfig API. It will break existing
configurations if they run it through authconfig. (SSSDConfig API will
throw an exception on the unknown value).
If I'm reading this right, a user is going to see the deprecation error
message if they have neither krb5_kdcip nor krb5_server specified. This
is a bug, since if neither is specified, we use service discovery.
confdb_get_string will return EOK and set krb5_servers to NULL if
krb5_kdcip is not found. You should handle this case accordingly.
- --
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/
iEYEARECAAYFAkyjJLoACgkQeiVVYja6o6M3hQCePcQ92NZqea2J1pG5B7qDBEVE
PIUAn1JfP8GyDsUg5j4eJpucMHs5CYCv
=Lk1q
-----END PGP SIGNATURE-----
More information about the sssd-devel
mailing list