On Tue, Feb 09, 2016 at 08:37:04AM +0100, Lukas Slebodnik wrote:
> On (09/02/16 08:17), Jakub Hrozek wrote:
> >On Fri, Jan 29, 2016 at 02:30:36PM +0100, Pavel Reichl wrote:
> >> Hello, please see trivial patch attached. Thanks.
> >
> >> From 6d5f6b71c2d2f891470dc1c9f08ae67f5b6c02f5 Mon Sep 17 00:00:00 2001
> >> From: Pavel Reichl <[email protected]>
> >> Date: Fri, 29 Jan 2016 08:27:01 -0500
> >> Subject: [PATCH] PAM: Clarify man page for domains option
> >> 
> >> Resolves:
> >> https://fedorahosted.org/sssd/ticket/2946
> >> ---
> >>  src/man/pam_sss.8.xml | 8 +++++---
> >>  1 file changed, 5 insertions(+), 3 deletions(-)
> >> 
> >> diff --git a/src/man/pam_sss.8.xml b/src/man/pam_sss.8.xml
> >> index 
> >> 7794d3acfdfdbde491a3e1ada44481b73588e41f..278126c14d0a574a1e120762af264ef653deb0b0
> >>  100644
> >> --- a/src/man/pam_sss.8.xml
> >> +++ b/src/man/pam_sss.8.xml
> >> @@ -145,9 +145,11 @@
> >>                          SSSD domain names, as specified in the sssd.conf 
> >> file.
> >>                      </para>
> >>                      <para>
> >> -                        NOTE: Must be used in conjunction with the
> >> -                        <quote>pam_trusted_users</quote> and
> >> -                        <quote>pam_public_domains</quote> options.
> >> +                        NOTE: If PAM service is being run by untrusted 
> >> user
> >> +                        (<quote>pam_trusted_users</quote> option)
> >> +                        then please make
> >> +                        sure that restricted domains are public
> >> +                        (<quote>pam_public_domains</quote> option).
> >>                          Please see the
> >>                          <citerefentry>
> >>                              <refentrytitle>sssd.conf</refentrytitle>
> >> -- 
> >> 2.4.3
> >> 
> >
> >I'm sorry, but this doesn't read any better to me. Especially I don't
> >understand "restricted domains are public", sounds like an oxymoron to
> >me.
> Should we ask Aneta for help/review ?

Maybe, but it was her who asked us to improve this paragraph in the
first place :)
_______________________________________________
sssd-devel mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]

Reply via email to