https://fedorahosted.org/freeipa/ticket/2620

When renaming object its case sensitivity is obeyed. This was DS bug.
Unit tests were corrected and minimal DS version was updated in spec
file.

The minimal version for F16 and lower is still in koji only, so we may wait
with pushing these patch.

--
Regards,

Ondrej Hamada
FreeIPA team
jabber: oh...@jabbim.cz
IRC: ohamada

From bbfbab611358136c4afe57e458c86de0e1c4b956 Mon Sep 17 00:00:00 2001
From: Ondrej Hamada <oham...@redhat.com>
Date: Tue, 19 Jun 2012 14:49:19 +0200
Subject: [PATCH] Case sensitive renaming of objects

When renaming object its case sensitivity is obeyed. This was DS bug.
Unit tests were corrected and minimal DS version was updated in spec
file.

https://fedorahosted.org/freeipa/ticket/2620
---
 freeipa.spec.in                             |   10 +++++++++-
 tests/test_xmlrpc/test_permission_plugin.py |    8 ++++----
 2 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/freeipa.spec.in b/freeipa.spec.in
index 64abcc9f972d565c1061264f837f5664ac3abb2a..7cda33c5fa07048a358a0e3396922c2dd09301e5 100644
--- a/freeipa.spec.in
+++ b/freeipa.spec.in
@@ -96,7 +96,11 @@ Requires: %{name}-python = %{version}-%{release}
 Requires: %{name}-client = %{version}-%{release}
 Requires: %{name}-admintools = %{version}-%{release}
 Requires: %{name}-server-selinux = %{version}-%{release}
-Requires(pre): 389-ds-base >= 1.2.10.4-2
+%if 0%{?fedora} >= 17
+Requires(pre): 389-ds-base >= 1.2.11.5-1
+%else
+Requires(pre): 389-ds-base >= 1.2.10.10-1
+%endif
 Requires: openldap-clients
 Requires: nss
 Requires: nss-tools
@@ -713,6 +717,10 @@ fi
 %ghost %attr(0644,root,apache) %config(noreplace) %{_sysconfdir}/ipa/ca.crt
 
 %changelog
+* Wed Jun 20 2012 Ondrej Hamada <oham...@redhat.com> - 2.99.0-33
+- Set min for 389-ds-base to 1.2.11.5-1 on F17 to fix installation issue
+- Set min for 389-ds-base to 1.2.10.10-1 on F16 (and lower) to fix CN case persistence
+
 * Fri Jun 8 2012 Martin Kosek <mko...@redhat.com> - 2.99.0-32
 - Add directory /var/lib/ipa/sysupgrade for package upgrade metadata
 - Set min for bind-dyndb-ldap to 1.1.0-0.12.rc1 to pick up persistent search
diff --git a/tests/test_xmlrpc/test_permission_plugin.py b/tests/test_xmlrpc/test_permission_plugin.py
index 6613c9bbac985dd5732e14413cbe46135a789d55..c88e086fa06ebe82e1fc343d008b9670dc38bb86 100644
--- a/tests/test_xmlrpc/test_permission_plugin.py
+++ b/tests/test_xmlrpc/test_permission_plugin.py
@@ -37,7 +37,7 @@ permission1_renamed_dn = DN(('cn',permission1_renamed),
                             api.env.container_permission,api.env.basedn)
 
 permission1_renamed_ucase = u'Testperm_RN'
-permission1_renamed_ucase_dn = DN(('cn',permission1_renamed_ucase.lower()),
+permission1_renamed_ucase_dn = DN(('cn',permission1_renamed_ucase),
                             api.env.container_permission,api.env.basedn)
 
 
@@ -592,7 +592,7 @@ class test_permission(Declarative):
                 summary=u'Modified permission "%s"' % permission1_renamed,
                 result={
                     'dn': lambda x: DN(x) == permission1_renamed_ucase_dn,
-                    'cn': [permission1_renamed_ucase.lower()],
+                    'cn': [permission1_renamed_ucase],
                     'member_privilege': [privilege1],
                     'type': u'user',
                     'permissions': [u'write'],
@@ -612,7 +612,7 @@ class test_permission(Declarative):
                 summary=u'Modified permission "%s"' % permission1_renamed_ucase,
                 result=dict(
                     dn=lambda x: DN(x) == permission1_renamed_ucase_dn,
-                    cn=[permission1_renamed_ucase.lower()],
+                    cn=[permission1_renamed_ucase],
                     member_privilege=[privilege1],
                     subtree=u'ldap:///cn=*,cn=test,cn=accounts,%s' % api.env.basedn,
                     permissions=[u'write'],
@@ -632,7 +632,7 @@ class test_permission(Declarative):
                 result=[
                     {
                         'dn':lambda x: DN(x) == permission1_renamed_ucase_dn,
-                        'cn':[permission1_renamed_ucase.lower()],
+                        'cn':[permission1_renamed_ucase],
                         'member_privilege':[privilege1],
                         'subtree':u'ldap:///cn=*,cn=test,cn=accounts,%s' % api.env.basedn,
                         'permissions':[u'write'],
-- 
1.7.6.5

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to