Directory Studio
  1. Directory Studio
  2. DIRSTUDIO-250

Unable to load entries if RDN is quoted and contains unescaped comma

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.1
    • Fix Version/s: 1.1.0
    • Component/s: studio-ldapbrowser
    • Labels:
      None
    • Environment:
      WinXP SP2

      Description

      If an entry's RDN contains a comma, an error is generated if the RDN is quoted instead of the comma being escape. Ex:

      uid="test,123", ou=people, dc=example, dc=com - this generate an error
      uid=test\,123, ou=people, dc=example, dc=com - this works

      The error displayed is "Invalid RDN", and the ldap browser halts loading any more objects in that container.

        Activity

        Ron Frazier created issue -
        Hide
        Emmanuel Lecharny added a comment -

        I confirm this is a bug.

        Show
        Emmanuel Lecharny added a comment - I confirm this is a bug.
        Hide
        Stefan Seelmann added a comment -

        This and many other problems with RDNs and DNs has been fixed recently. We replaced the internal DN parser by the good shared-ldap parser (see DIRSTUDIO-229).

        Show
        Stefan Seelmann added a comment - This and many other problems with RDNs and DNs has been fixed recently. We replaced the internal DN parser by the good shared-ldap parser (see DIRSTUDIO-229 ).
        Stefan Seelmann made changes -
        Field Original Value New Value
        Resolution Fixed [ 1 ]
        Assignee Stefan Seelmann [ seelmann ]
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.1.0 [ 12312701 ]
        Hide
        Emmanuel Lecharny added a comment -

        You are right, Stefan. I have just checked that it has been fixed in the trunk, and it is. However, we didn't had specific tests for this case, so I have added one.

        Show
        Emmanuel Lecharny added a comment - You are right, Stefan. I have just checked that it has been fixed in the trunk, and it is. However, we didn't had specific tests for this case, so I have added one.
        Hide
        Stefan Seelmann added a comment -

        Thanks Emmanuel for verifying and for the test.

        Show
        Stefan Seelmann added a comment - Thanks Emmanuel for verifying and for the test.
        Hide
        Pierre-Arnaud Marcelot added a comment -

        Version 1.1.0 has been released.
        Closing this issue.

        Show
        Pierre-Arnaud Marcelot added a comment - Version 1.1.0 has been released. Closing this issue.
        Pierre-Arnaud Marcelot made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        3h 45m 1 Stefan Seelmann 11/Dec/07 19:32
        Resolved Resolved Closed Closed
        184d 17h 21m 1 Pierre-Arnaud Marcelot 13/Jun/08 13:54

          People

          • Assignee:
            Stefan Seelmann
            Reporter:
            Ron Frazier
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development