Uploaded image for project: 'Directory Client API'
  1. Directory Client API
  2. DIRAPI-192

Rdn.escapeValue() should not escape unicode characters

        Details

        • Type: Bug
        • Status: Closed
        • Priority: 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

            blocks

            Bug - A problem which impairs or prevents the functions of the product. DIRSTUDIO-963 Why UTF-8 is escaped in DN strings since 2.0.0?

            • Major - Major loss of function.
            • Resolved

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              seelmann 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
              seelmann 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
              Permalink
              elecharny 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
              elecharny 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
              Permalink
              seelmann Stefan Seelmann added a comment -

              Fixed here http://svn.apache.org/r1596674

              Show
              seelmann Stefan Seelmann added a comment - Fixed here http://svn.apache.org/r1596674
              Hide
              Permalink
              elecharny Emmanuel Lecharny added a comment -

              Closing the resolved issues.

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

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development