diff options
author | Martin Kosek <mkosek@redhat.com> | 2013-01-18 17:33:19 +0100 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2013-02-14 08:38:11 +0100 |
commit | d79aac855b31523b8dd7efcfd0bea3feb59cdaa0 (patch) | |
tree | 04a0f647be959de0834412b420d87aa0ce5276b9 /ipalib/plugins/group.py | |
parent | e60e80e2b6710e581e417d9e7e05cea21ba9f6b0 (diff) | |
download | freeipa-d79aac855b31523b8dd7efcfd0bea3feb59cdaa0.tar.gz freeipa-d79aac855b31523b8dd7efcfd0bea3feb59cdaa0.tar.xz freeipa-d79aac855b31523b8dd7efcfd0bea3feb59cdaa0.zip |
Do not hide SID resolver error in group-add-member
When group-add-member does not receive any resolved trusted domain
object SID, it raises an exception which hides any useful error
message passed by underlying resolution methods. Remove the exception
to reveal this error messages to user.
https://fedorahosted.org/freeipa/ticket/2997
Diffstat (limited to 'ipalib/plugins/group.py')
-rw-r--r-- | ipalib/plugins/group.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/ipalib/plugins/group.py b/ipalib/plugins/group.py index 347a7ee9f..19404c6fa 100644 --- a/ipalib/plugins/group.py +++ b/ipalib/plugins/group.py @@ -390,9 +390,6 @@ class group_add_member(LDAPAddMember): failed_sids.append((sid, unicode(e))) else: sids.append(actual_sid) - if len(sids) == 0: - raise errors.ValidationError(name=_('external member'), - error=_('values are not recognized as valid SIDs from trusted domain')) restore = [] if 'member' in failed and 'group' in failed['member']: restore = failed['member']['group'] |