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

          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.
          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
          Hide
          Emmanuel Lecharny added a comment -

          Closing the resolved issues.

          Show
          Emmanuel Lecharny added a comment - Closing the resolved issues.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development