URL: https://github.com/SSSD/sssd/pull/911
Title: #911: Update pam_sss.8.xml

sumit-bose commented:
"""
Hi,

thank you for the patch.

you wording is mainly about authentication, but pam_sss handles authorization 
and session setup as well. Maybe you can check if you can make at least some of 
the descriptions more general.

You didn't mention return codes like e.g. `PAM_PERM_DENIED` or `PAM_AUTH_ERR`. 
Is this because you didn't find a good description or are you planning to add 
them in a later pull-request?

bye,
Sumit
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/911#issuecomment-546306056
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org

Reply via email to