Supported providers are mentioned in Description section, because there was already partial description present. Another possibility was to comment it as configuration option (e.g. create access_provider=ldap record there). I like this way more, but I'm not sure whether it is descriptive enough.
-- Jan
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 09/13/2010 04:59 AM, Jan Zelený wrote:
Supported providers are mentioned in Description section, because there was already partial description present. Another possibility was to comment it as configuration option (e.g. create access_provider=ldap record there). I like this way more, but I'm not sure whether it is descriptive enough.
-- Jan
Nack.
I don't really like that we're explicitly mentioning that there is no access provider for Kerberos (since we have https://fedorahosted.org/sssd/ticket/618 open to create one). Can you change it to be just a positive list of the providers it currently supports? (auth and chpass).
Also, you should mention the chpass providers for both IPA and LDAP as well.
- -- Stephen Gallagher RHCE 804006346421761
Delivering value year after year. Red Hat ranks #1 in value among software vendors. http://www.redhat.com/promo/vendor/
Stephen Gallagher sgallagh@redhat.com wrote:
On 09/13/2010 04:59 AM, Jan Zelený wrote:
Supported providers are mentioned in Description section, because there was already partial description present. Another possibility was to comment it as configuration option (e.g. create access_provider=ldap record there). I like this way more, but I'm not sure whether it is descriptive enough.
Nack.
I don't really like that we're explicitly mentioning that there is no access provider for Kerberos (since we have https://fedorahosted.org/sssd/ticket/618 open to create one). Can you change it to be just a positive list of the providers it currently supports? (auth and chpass).
Also, you should mention the chpass providers for both IPA and LDAP as well.
Updated patch is in attachment.
Jan
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 09/29/2010 06:40 AM, Jan Zelený wrote:
Stephen Gallagher sgallagh@redhat.com wrote:
On 09/13/2010 04:59 AM, Jan Zelený wrote:
Supported providers are mentioned in Description section, because there was already partial description present. Another possibility was to comment it as configuration option (e.g. create access_provider=ldap record there). I like this way more, but I'm not sure whether it is descriptive enough.
Nack.
I don't really like that we're explicitly mentioning that there is no access provider for Kerberos (since we have https://fedorahosted.org/sssd/ticket/618 open to create one). Can you change it to be just a positive list of the providers it currently supports? (auth and chpass).
Also, you should mention the chpass providers for both IPA and LDAP as well.
Updated patch is in attachment.
Jan
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 SIGNED MESSAGE----- Hash: SHA1
On 10/04/2010 07:38 AM, Stephen Gallagher wrote:
On 09/29/2010 06:40 AM, Jan Zelený wrote:
Stephen Gallagher sgallagh@redhat.com wrote:
On 09/13/2010 04:59 AM, Jan Zelený wrote:
Supported providers are mentioned in Description section, because there was already partial description present. Another possibility was to comment it as configuration option (e.g. create access_provider=ldap record there). I like this way more, but I'm not sure whether it is descriptive enough.
Nack.
I don't really like that we're explicitly mentioning that there is no access provider for Kerberos (since we have https://fedorahosted.org/sssd/ticket/618 open to create one). Can you change it to be just a positive list of the providers it currently supports? (auth and chpass).
Also, you should mention the chpass providers for both IPA and LDAP as well.
Updated patch is in attachment.
Jan
Ack.
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/
sssd-devel@lists.fedorahosted.org