Univention Bugzilla – Full Text Bug Listing |
Summary: | univention.uldap.access.rename() wrong DN syntax handling | ||
---|---|---|---|
Product: | UCS | Reporter: | Florian Best <best> |
Component: | univention-lib | Assignee: | Florian Best <best> |
Status: | CLOSED FIXED | QA Contact: | Philipp Hahn <hahn> |
Severity: | normal | ||
Priority: | P5 | CC: | gohmann |
Version: | UCS 4.2 | Keywords: | interim-2 |
Target Milestone: | UCS 4.2 | ||
Hardware: | Other | ||
OS: | Linux | ||
What kind of report is it?: | Development Internal | What type of bug is this?: | --- |
Who will be affected by this bug?: | --- | How will those affected feel about the bug?: | --- |
User Pain: | Enterprise Customer affected?: | ||
School Customer affected?: | ISV affected?: | ||
Waiting Support: | Flags outvoted (downgraded) after PO Review: | ||
Ticket number: | Bug group (optional): | ||
Max CVSS v3 score: | |||
Bug Depends on: | 37337 | ||
Bug Blocks: | 43430, 43431 |
Description
Florian Best
2017-01-12 18:35:54 CET
r75728 | Changelog Bug #43332 univention-python (10.0.2-1): r75726 | Bug #43332: fix DN syntax when renamed object contains "," OK: r75728 r75726 OK: udm container/cn create --set name='foo bar' udm container/cn modify --dn 'cn=foo bar,$LB' --set name='foo, bar' OK: /usr/share/ucs-test/63_udm-containers/17_container_cn_rename_uppercase The test case revealed one more error during move: # udm container/ou create --set 'name=âeg;=_,~~3' Object created: ou=âeg\;\=_\,~~3,dc=school,dc=local # udm container/ou create --set 'name=ômt+€|,"-8' Object created: ou=ômt\+€|\,\"-8,dc=school,dc=local # udm container/ou move --dn "ou=âeg\;\=_\,~~3,$(ucr get ldap/base)" --position "ou=ômt\+€|\,\"-8,(ucr get ldap/base)" Object modified: ou=âeg\;\=_\,~~3,dc=school,dc=local # univention-ldapsearch -LLL 'ou=*' dn | ldapsearch-wrapper | ldapsearch-decode64 → object was moved to a wrong position/renamed. univention-directory-manager-modules (12.0.12-1): r76019 | Bug #43332: fix moving of objects containing "," The tests reveal another error if you modify the name of a container/ou which has objects underneath of it: # eval "$(ucr shell)" # udm container/ou create --position "$ldap_base" --set 'name=ŝnfĉüß-|~ê' Object created: ou=ŝnfĉüß-|~ê,dc=school,dc=local # udm users/user create --position "ou=ŝnfĉüß-|~ê,$ldap_base" --set username=Xcn3zxg5ouz --set password=univention --set firstname=oyri1fxljy --set lastname=g3r74gcaon Object created: uid=Xcn3zxg5ouz,ou=ŝnfĉüß-|~ê,dc=school,dc=local # udm container/ou modify --dn "ou=ŝnfĉüß-|~ê,$ldap_base" --set 'name=ŝNFĉüß-|~ê' Object modified: ou=ŝNFĉüß-|~ê,ou=temporary_move_container_1485258882.38,dc=school,dc=local → It shows the DN of a temporary OU containter univention-directory-manager-modules (12.0.12-2): r76054 | Bug #43332: fix return value of object.move() which supports subtree_move and has sub objects OK: r75728 r75726 r76019 r76054 OK: LB=$(ucr get ldap/base) udm container/cn create --set name='a' udm container/cn modify --dn "cn=a,$LB" --set name='b,c' # 4.1: LDAP Error: Invalid DN syntax: invalid new RDN # 4.2: Object modified: cn=b\,c,$LB OK: LB=$(ucr get ldap/base) udm container/cn create --set name='a,b' udm container/cn create --set name='x,y' udm container/cn move --dn "cn=a\,b,$LB" --position "cn=x\,y,$LB" # 4.1: ldap Error: Invalid DN syntax: invalid new RDN # 4.2: Object modified: cn=a\,b,$LB OK: LB=$(ucr get ldap/base) udm container/cn create --set name='a,b' udm container/cn create --position "cn=a\,b,$LB" --set name='x,y' udm container/cn modify --dn "cn=a\,b,$LB" --set name='p,q' # 4.1: LDAP Error: Invalid DN syntax: invalid new RDN # 4.2: Object modified: cn=p\,q,dc=phahn-dev,dc=intranet OK: LB=$(ucr get ldap/base) udm container/cn create --set name='a,b' udm container/cn create --position "cn=a\,b,$LB" --set name='x,y' udm container/cn modify --dn "cn=a\,b,$LB" --set name='A,B' OK: please fix those too: r76143 r76144 $ git grep -n "[Dd][Nn].*\.find([\"'],[\"'])" management/univention-directory-manager-modules/modules/univention/admin/handlers/__init__.py:412: raise univention.admin.uexceptions.invalidOperation(_('Unable to move object %(name)s (%(type)s) in subtree, trying to revert changes.') % {'name': subolddn[:subolddn.find(',')], 'type': univention.admin.modules.identifyOne(subolddn, suboldattrs)}) management/univention-directory-manager-modules/modules/univention/admin/handlers/__init__.py:456: raise univention.admin.uexceptions.invalidOperation(_('Unable to move object %(name)s (%(type)s) in subtree, trying to revert changes.') % {'name': subolddn[:subolddn.find(',')], 'type': univention.admin.modules.identifyOne(subolddn, suboldattrs)}) management/univention-directory-manager-modules/modules/univention/admin/handlers/groups/group.py:818: # newuids = map(lambda x: x[x.find('=') + 1: x.find(',')], newmembers) UCS 4.2 has been released: https://docs.software-univention.de/release-notes-4.2-0-en.html https://docs.software-univention.de/release-notes-4.2-0-de.html If this error occurs again, please use "Clone This Bug". |