You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@directory.apache.org by "Stefan Seelmann (JIRA)" <ji...@apache.org> on 2015/06/15 23:41:00 UTC
[jira] [Comment Edited] (DIRKRB-293) Implementing doUpdateIdentity
for LdapIdentityBackend
[ https://issues.apache.org/jira/browse/DIRKRB-293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14586734#comment-14586734 ]
Stefan Seelmann edited comment on DIRKRB-293 at 6/15/15 9:40 PM:
-----------------------------------------------------------------
Hi Yaning, here are some suggestions (which also apply to DIRKRB-291, DIRKRB-292 and DIRKRB-294)
* Please don't construct a DN by concatenating strings. if the UID contains special characters it may produce an invalid DN. Instead you can use {{new Dn(new Rdn("uid", uid), BASE_DN)}}.
* Please define and use constants for the LDAP attributes. KerberosAttribute in ApacheDS kerberos-codec already defines them, so you can copy them from there.
* Instead of repeated {{new DefaultModification()}} -please try to create a helper method- you can try to use ModifyRequest/ModifyRequestImpl and its replace method.
was (Author: seelmann):
Hi Yaning, here are some suggestions (which also apply to DIRKRB-291, DIRKRB-292 and DIRKRB-294)
* Please don't construct a DN by concatenating strings. if the UID contains special characters it may produce an invalid DN. Instead you can use {{new Dn(new Rdn("uid", uid), BASE_DN)}}.
* Please define and use constants for the LDAP attributes. KerberosAttribute in ApacheDS kerberos-codec already defines them, so you can copy them from there.
* Instead of repeated {{new DefaultModification()}} please try to create a helper method.
> Implementing doUpdateIdentity for LdapIdentityBackend
> -----------------------------------------------------
>
> Key: DIRKRB-293
> URL: https://issues.apache.org/jira/browse/DIRKRB-293
> Project: Directory Kerberos
> Issue Type: Sub-task
> Reporter: Xu Yaning
> Assignee: Xu Yaning
> Attachments: DIRKRB-293-v1.patch
>
>
> Impelment {{doUpdateIdentity}} for {{LdapIdentityBackend}} to update an identity.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)