Bug 1250177 - Investigate betxn plugins to ensure they return the correct error code
Summary: Investigate betxn plugins to ensure they return the correct error code
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: 389-ds-base
Version: 7.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: ---
Assignee: Noriko Hosoi
QA Contact: Viktor Ashirov
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-08-04 16:29 UTC by Noriko Hosoi
Modified: 2020-09-13 21:06 UTC (History)
4 users (show)

Fixed In Version: 389-ds-base-1.3.4.0-9.el7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-11-19 11:43:56 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github 389ds 389-ds-base issues 1141 0 None None None 2020-09-13 21:06:30 UTC
Red Hat Product Errata RHBA-2015:2351 0 normal SHIPPED_LIVE 389-ds-base bug fix and enhancement update 2015-11-19 10:28:44 UTC

Description Noriko Hosoi 2015-08-04 16:29:43 UTC 
This bug is created as a clone of upstream ticket:
https://fedorahosted.org/389/ticket/47810

Some betxn plugins did not return error codes(they always return success).  This was because before there were betxn plugins, their returns codes did not affect the if the operation was accepted or aborted.  Now those errors do matter, and we need to make sure all the plugins are behaving correctly.
Comment 1 Noriko Hosoi 2015-08-04 16:31:53 UTC 
Bug Description:  When the memberOf plugin tries to add memberOf attribute to
                  an entry during a mod-replace on a group, even though the
                  update to the user entry fails, but plugin still allows
                  the member to be added to the group.
Comment 2 Jenny Severance 2015-08-04 17:02:11 UTC 
Please add steps to reproduce.
Comment 3 mreynolds 2015-08-04 18:46:23 UTC 
script to verify fix:

ds/dirsrvtests/suites/betxn/betxn_test.py

Fixed upstream.
Comment 5 Viktor Ashirov 2015-08-19 09:01:34 UTC 
Build tested: 389-ds-base-1.3.4.0-12.el7.x86_64

============================================================================ test session starts =============================================================================
platform linux2 -- Python 2.7.5 -- py-1.4.30 -- pytest-2.7.2 -- /usr/bin/python
rootdir: /tmp/test/ds/dirsrvtests/suites/betxns, inifile: 
collected 5 items 

../tmp/test/ds/dirsrvtests/suites/betxns/betxn_test.py::test_betxn_init PASSED
../tmp/test/ds/dirsrvtests/suites/betxns/betxn_test.py::test_betxt_7bit PASSED
../tmp/test/ds/dirsrvtests/suites/betxns/betxn_test.py::test_betxn_attr_uniqueness PASSED
../tmp/test/ds/dirsrvtests/suites/betxns/betxn_test.py::test_betxn_memberof PASSED
../tmp/test/ds/dirsrvtests/suites/betxns/betxn_test.py::test_betxn_final PASSED

========================================================================= 5 passed in 30.52 seconds ==========================================================================

Marking as VERIFIED.
Comment 6 errata-xmlrpc 2015-11-19 11:43:56 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2015-2351.html
Note You need to log in before you can comment on or make changes to this bug.