Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-3199

Avoid error logs for invalid jcr paths in AggregatorFilter

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • 4.3.2
    • 4.3.1
    • None
    • None

      Since we now allow extended chars for any url, Aggregator filter may now check the esistence in the jcr repository of a path containing invalid chars.

      Although this doesn't cause any problem to the user (an invalid path is translated to a 404, which is the expected result), this may generate some annoying logs.

      Requesting a URL with:

      generates a MalformedPath/IllegalName exception wrapped into a RepositoryException and logged (with error level).

      Since there is no specific exception thrown we can't add an appropriate silent catch, so we should avoid calling isExist() for invalid handles.

      Unfortunately it seems there is nothing available for validating a path in Jcr... Jackrabbit has something similar but only in version 2.0 and using jackrabbit-specific APIs, so it's not an option. The only simple solution is to explicitly check for common chars/rules like the ones before in the Aggregator filter.

        Acceptance criteria

              fgiust Fabrizio Giustina
              fgiust Fabrizio Giustina
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Bug DoR
                  Task DoD