[SSSD] [PATCH] LDAP: warn about lockout option being deprecated
Jakub Hrozek
jhrozek at redhat.com
Thu May 14 13:31:55 UTC 2015
On Tue, May 12, 2015 at 03:42:47PM +0200, Pavel Reichl wrote:
> >>>> From 3adee0c2ff207eef7c2edcfed80eb47754570210 Mon Sep 17 00:00:00 2001
> >>>>From: Pavel Reichl <preichl at redhat.com>
> >>>>Date: Wed, 25 Mar 2015 05:03:12 -0400
> >>>>Subject: [PATCH] LDAP: warn about lockout option being deprecated
> >>>>
> >>>>---
> >>>> src/man/sssd-ldap.5.xml | 7 +++++++
> >>>> src/providers/ldap/sdap_access.c | 9 ++++++++-
> >>>> 2 files changed, 15 insertions(+), 1 deletion(-)
> >>>>
> >>>>diff --git a/src/man/sssd-ldap.5.xml b/src/man/sssd-ldap.5.xml
> >>>>index 00da3964af7619f19525e76c4f1292586dd60f54..b46e07fec96ffa7305ab41ce42c16b2e94e53292 100644
> >>>>--- a/src/man/sssd-ldap.5.xml
> >>>>+++ b/src/man/sssd-ldap.5.xml
> >>>>@@ -1955,6 +1955,13 @@ ldap_access_filter = (employeeType=admin)
> >>>> be set for this feature to work.
> >>>> </para>
> >>>> <para>
> >>>>+ <emphasis>
> >>>>+ Please note that this option is superseded by
> >>>>+ ppolicy option and will be dropped in next major
> >>>>+ release.
> >>>Since we can't remove the option soon, I'd say "might be removed in a
> >>>future release". It would also be nice to point the admin to the option
> >>>he should be using now instead.
> >>OK, I mentioned ppolicy option for the second time, or did you just want me
> >>to rephrase the sentence?
> >Ah, sorry, no. I was reading the rendered text and didn't realize
> >ppolicy was an option and not a generic term. Sorry, can you remove the
> >added sentence adn then just enclose ppolicy in <quote> ? That would
> >make the rendering better readable I guess. Also, "the ppolicy option".
> OK, please see updated patch.
ACK
More information about the sssd-devel
mailing list