You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@felix.apache.org by "Carsten Ziegeler (JIRA)" <ji...@apache.org> on 2015/03/17 08:00:53 UTC

[jira] [Closed] (FELIX-2843) Parsing PID assumes last 4 chars are file extension

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

Carsten Ziegeler closed FELIX-2843.
-----------------------------------

> Parsing PID assumes last 4 chars are file extension
> ---------------------------------------------------
>
>                 Key: FELIX-2843
>                 URL: https://issues.apache.org/jira/browse/FELIX-2843
>             Project: Felix
>          Issue Type: Bug
>          Components: File Install
>    Affects Versions: fileinstall-3.1.10
>            Reporter: Carl Hall
>            Assignee: Carsten Ziegeler
>            Priority: Blocker
>             Fix For: fileinstall-3.2.0
>
>         Attachments: felix-2843.diff
>
>
> In ConfigInstaller.parsePid(String), to clear off the path extension the following logic is used.
> String pid = path.substring(0, path.length() - 4);
> This appears to be a leftover from when only .cfg (4 chars) was allowed as a file extension. Since .config is now a valid file extension, the above logic leaves ".co" on the end of the service pid that is parsed out.
> This blocks configuring of services when a configuration file ending with .config is used. .config syntax is required if a property needs to be something like String[].



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)