[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