URL: https://github.com/SSSD/sssd/pull/242
Title: #242: IPA: Improve DEBUG message if an external group has no SID

jhrozek commented:
"""
On Fri, Apr 21, 2017 at 04:22:44AM -0700, lslebodn wrote:
> lslebodn commented on this pull request.
> 
> 
> 
> > @@ -1308,7 +1308,10 @@ static void ipa_s2n_get_list_next(struct tevent_req 
> > *subreq)
>      ret = sysdb_attrs_get_string(state->attrs->sysdb_attrs, SYSDB_SID_STR,
>                                   &sid_str);
>      if (ret != EOK) {
> -        DEBUG(SSSDBG_OP_FAILURE, "sysdb_attrs_get_string failed.\n");
> +        DEBUG(SSSDBG_CRIT_FAILURE,
> +              "Group [%s] has no SID, please check the "
> 
> Are we sure that it cannot be user but only group?

Since the request accepts a type, which can be something else than
BE-REQ_GROUP, I updated the message to say 'Object'.

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/242#issuecomment-296170329
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

Reply via email to