diff options
author | Kevin McCarthy <kmccarth@redhat.com> | 2007-08-17 15:32:05 -0700 |
---|---|---|
committer | Kevin McCarthy <kmccarth@redhat.com> | 2007-08-17 15:32:05 -0700 |
commit | 5f0f192ae3c8aa44868b7b3b86e71523fc55515b (patch) | |
tree | a99a70cd9ce3a93feef7562f42f6fa5dda1135f0 | |
parent | a0e2fa00f1005bf7d5853c83ff0f567663352e92 (diff) | |
download | freeipa.git-5f0f192ae3c8aa44868b7b3b86e71523fc55515b.tar.gz freeipa.git-5f0f192ae3c8aa44868b7b3b86e71523fc55515b.tar.xz freeipa.git-5f0f192ae3c8aa44868b7b3b86e71523fc55515b.zip |
change strings to be encoded using value.encode("utf-8").
fix update to set 'cn' instead of 'sn'.
remove str() in funcs.add_user().
-rw-r--r-- | ipa-server/ipa-gui/ipagui/controllers.py | 4 | ||||
-rw-r--r-- | ipa-server/xmlrpc-server/funcs.py | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/ipa-server/ipa-gui/ipagui/controllers.py b/ipa-server/ipa-gui/ipagui/controllers.py index ae8b088b..36c1bf84 100644 --- a/ipa-server/ipa-gui/ipagui/controllers.py +++ b/ipa-server/ipa-gui/ipagui/controllers.py @@ -49,7 +49,7 @@ def set_ldap_value(hash, key, value): """Converts unicode strings to normal strings (because LDAP is choking on unicode strings""" if value != None: - value = str(value) + value = value.encode('utf-8') hash[key] = value @@ -137,7 +137,7 @@ class Root(controllers.RootController): # # this is a hack until we decide on the policy for names/cn/sn/givenName # - set_ldap_value(new_user, 'sn', + set_ldap_value(new_user, 'cn', "%s %s" % (kw.get('givenname'), kw.get('sn'))) orig_user = to_ldap_hash(orig_user) diff --git a/ipa-server/xmlrpc-server/funcs.py b/ipa-server/xmlrpc-server/funcs.py index 22f52dae..a182d4b2 100644 --- a/ipa-server/xmlrpc-server/funcs.py +++ b/ipa-server/xmlrpc-server/funcs.py @@ -197,7 +197,7 @@ class IPAServer: user = user[0] dn="uid=%s,%s,%s" % (user['uid'], user_container,self.basedn) - entry = ipaserver.ipaldap.Entry(str(dn)) + entry = ipaserver.ipaldap.Entry(dn) # some required objectclasses entry.setValues('objectClass', 'top', 'posixAccount', 'shadowAccount', 'account', 'person', 'inetOrgPerson', 'organizationalPerson', 'krbPrincipalAux', 'krbTicketPolicyAux') @@ -214,7 +214,7 @@ class IPAServer: # fill in our new entry with everything sent by the user for u in user: - entry.setValues(str(u), str(user[u])) + entry.setValues(u, user[u]) if opts: self.set_principal(opts['remoteuser']) |