[SSSD] [sssd PR#242][comment] IPA: Improve DEBUG message if an external group has no SID

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

jhrozek commented:
"""
* master: ef019268d2d112ebff3577e551cd19478d73d93b
"""

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


[SSSD] [sssd PR#242][comment] IPA: Improve DEBUG message if an external group has no SID

2017-04-21 Thread jhrozek
  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