[SSSD] [PATCH] LDAP: warn about lockout option being deprecated

Jakub Hrozek jhrozek at redhat.com
Thu May 14 14:44:03 UTC 2015


On Thu, May 14, 2015 at 03:31:55PM +0200, Jakub Hrozek wrote:
> 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

* master: 108a49f0e816d95cf75a1e964f63b397e53c8b56
* sssd-1-12: 1590f8dbcbfc0a3c7a858590e54678ac7784be49



More information about the sssd-devel mailing list