Mailing-List: contact dev-help@directory.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Apache Directory Developers List" <dev@directory.apache.org>
Date: Wed, 18 May 2016 09:03:12 +0000 (UTC)
From: "Emmanuel Lecharny (JIRA)" <jira@apache.org>
To: dev@directory.apache.org
Message-ID: <JIRA.12970684.1463562164000.221730.1463562192925@Atlassian.JIRA>
In-Reply-To: <JIRA.12970684.1463562164000@Atlassian.JIRA>
References: <JIRA.12970684.1463562164000@Atlassian.JIRA> <JIRA.12970684.1463562164886@arcas>
Subject: [jira] [Created] (DIRAPI-283) We don't need to parse the DN when
 storing a name in the LDAP requests
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Wed, 18 May 2016 09:03:15 -0000

Emmanuel Lecharny created DIRAPI-283:
----------------------------------------

             Summary: We don't need to parse the DN when storing a name in the LDAP requests
                 Key: DIRAPI-283
                 URL: https://issues.apache.org/jira/browse/DIRAPI-283
             Project: Directory Client API
          Issue Type: Improvement
    Affects Versions: 1.0.0-M33
            Reporter: Emmanuel Lecharny
             Fix For: 2.0.0


When we store a name in a LDAP request, we usually parse it to see if it's a valid DN. For instance, in the {{BindRequestImpl}} class :

{noformat}
    /**
     * {@inheritDoc}
     */
    public BindRequest setName( String name )
    {
        this.name = name;

        try
        {
            this.dn = new Dn( name );
        }
        catch ( LdapInvalidDnException e )
        {
            // This might still be a valid DN (Windows AD binding for instance)
            LOG.debug( "Unable to convert the name to a DN." );
            this.dn = null;
        }

        return this;
    }
{noformat}

which get called in the {{StoreName}} method :

{noformat}
    /**
     * {@inheritDoc}
     */
    public void action( LdapMessageContainer<BindRequestDecorator> container ) throws DecoderException
    {
        BindRequest bindRequestMessage = container.getMessage();

        // Get the Value and store it in the BindRequest
        TLV tlv = container.getCurrentTLV();

        // We have to handle the special case of a 0 length name
        if ( tlv.getLength() == 0 )
        {
            bindRequestMessage.setName( "" );
        }
        else
        {
            byte[] nameBytes = tlv.getValue().getData();
            String nameStr = Strings.utf8ToString( nameBytes );

            try
            {
                // Testing the name as a DN
                new Dn( nameStr );
                bindRequestMessage.setName( nameStr );
            }
            catch ( LdapInvalidDnException ine )
            {
                String msg = "Incorrect DN given : " + nameStr + " (" + Strings.dumpBytes( nameBytes )
                    + ") is invalid";
...
{noformat}

As we can see, we first try to parse the {{DN}}, then we call the {{setName}} method with the String, and this method will parse the {{DN}} again...

Even worse : on the server, we need a schema aware version of the DN, which means we process the {(DN}} again to apply the {{SchemaManager}} on it.

That is clearly a waste of CPU : it's for the server to check the {{DN}}, this is not the decoder role.

For all the other request, we are checking if the {{DN}} is valid, which is already overdoing, but at least, we don't parse the {{DN}} twice.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)