Uploaded image for project: 'Jackrabbit Oak'
  1. Jackrabbit Oak
  2. OAK-5260

Incorrect handling of subpaths with leading left curly bracket

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.4.12, 1.5.17, 1.6.0
    • jcr
    • None

    Description

      As per SLING-6383 it looks like the Oak name mapping causes for example getItem("/libs/{sub") (with a left curly bracket in the path) to return the /libs node if that exists but the supplied path does not.

      I'll attach a patch with a test that demonstrates this.

      fmeschbe mentions in that Sling issue that the parsing of the CR 2 section 3.2.5.1 Expanded Form could be involved, treating the left curly bracket in a special way that's not appropriate here.

      Attachments

        1. OAK-5260.patch
          2 kB
          Bertrand Delacretaz
        2. OAK-5260-jsedding.patch
          4 kB
          Julian Sedding

        Issue Links

          blocks

          Bug - A problem which impairs or prevents the functions of the product. SLING-6383 Unexpected behavior with left curly bracket in resource resolution

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open
          is related to

          Bug - A problem which impairs or prevents the functions of the product. OAK-5410 Backport OAK-5260 (Incorrect handling of subpaths with leading left curly bracket)

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. OAK-5367 Strange path parsing

          • Major - Major loss of function.
          • Open
          relates to

          Task - A task that needs to be done. OAK-10596 Improve the test coverage of o.a.j.o.namepath.JcrPathParser

          • Major - Major loss of function.
          • Closed

          Activity

            People

              thomasm Thomas Mueller
              bdelacretaz Bertrand Delacretaz
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: