[jira] [Resolved] (SLING-6226) substVars not properly handling unknown properties

["Felix Meschberger (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/SLING-6226?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Felix Meschberger resolved SLING-6226.
--------------------------------------
    Resolution: Fixed

Thanks [~cziegeler].

Committed the patch in Rev. 1767618

> 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)