Directory Client API
  1. Directory Client API
  2. DIRAPI-192

Rdn.escapeValue() should not escape unicode characters

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.0.0-M22
    • Fix Version/s: 1.0.0-M23
    • Labels:
      None

      Description

      DIRSTUDIO-963 complains about escaped DNs. The reason is that we use Rdn.escapeValue() which escapes each multi-bye unicode character. I think according to RFC4514, section 2.4 this is not required, it just states "Other characters may be escaped".

        Issue Links

          Activity

          Stefan Seelmann created issue -
          Hide
          Stefan Seelmann added a comment - - edited

          Attached is a patch for discussion. Tests in shared and apacheds pass. The new constructors Rdn(Ava) is still work in progress.

          Show
          Stefan Seelmann added a comment - - edited Attached is a patch for discussion. Tests in shared and apacheds pass. The new constructors Rdn(Ava) is still work in progress.
          Stefan Seelmann made changes -
          Field Original Value New Value
          Attachment DIRAPI-192-shared.patch [ 12645597 ]
          Hide
          Emmanuel Lecharny added a comment -

          I think you are right. The grammar specifies that we can use any UTF-8 char, even if it's multibytes, and they don't need to be escaped.

          Let'w apply the patch if API and ApacheDS are passing.

          Show
          Emmanuel Lecharny added a comment - I think you are right. The grammar specifies that we can use any UTF-8 char, even if it's multibytes, and they don't need to be escaped. Let'w apply the patch if API and ApacheDS are passing.
          Hide
          Stefan Seelmann added a comment -
          Show
          Stefan Seelmann added a comment - Fixed here http://svn.apache.org/r1596674
          Stefan Seelmann made changes -
          Link This issue blocks DIRSTUDIO-963 [ DIRSTUDIO-963 ]
          Stefan Seelmann made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Emmanuel Lecharny added a comment -

          Closing the resolved issues.

          Show
          Emmanuel Lecharny added a comment - Closing the resolved issues.
          Emmanuel Lecharny made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Stefan Seelmann [ seelmann ] Emmanuel Lecharny [ elecharny ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          2d 2h 43m 1 Stefan Seelmann 21/May/14 21:51
          Resolved Resolved Closed Closed
          231d 20h 26m 1 Emmanuel Lecharny 08/Jan/15 17:17

            People

            • Assignee:
              Emmanuel Lecharny
              Reporter:
              Stefan Seelmann
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development