[
https://issues.apache.org/jira/browse/SLING-6226?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Felix Meschberger updated SLING-6226:
-------------------------------------
Attachment: SLING-6226.diff
Proposed patch adding the Util class, removing the old substVars
implementations and providing some test cases.
> substVars not properly handling unknown properties
> --------------------------------------------------
>
> Key: SLING-6226
> URL: https://issues.apache.org/jira/browse/SLING-6226
> Project: Sling
> Issue Type: Bug
> Affects Versions: Launchpad Base 2.6.14
> Reporter: Felix Meschberger
> Fix For: Launchpad Base 2.6.16
>
> Attachments: SLING-6226.diff
>
>
> Assuming a property in {{sling.properties}} of the form
> {code}
> org.apache.sling.commons.log.pattern=%d{yyyy-MM-dd HH:mm:ss.SSSXXX} *%level*
> [%thread] %logger %msg%n
> {code}
> This will break during Sling startup as the date format of this
> {{log.pattern}} ends with a curly brace but is not matched with a {{ $\{ }}.
> This is a but in the {{substVars}} method.
> Another problem is that this method is duplicate in the {{Sling}} and
> {{SlingServlet}} classes where the latter is in the web app class loader
> while the former already is in the launcher class loader.
> Proposing a patch taking the latest Felix Framework's Util class creating a
> new {{Util}} class in the shared package.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
