[jira] [Resolved] (SLING-4672) Set port via property file when launching standalone

["Carsten Ziegeler (JIRA)" <ji...@apache.org>]

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

Carsten Ziegeler resolved SLING-4672.
-------------------------------------
       Resolution: Fixed
    Fix Version/s: Launchpad Base 2.6.2

Thanks for your patch, Kevin. It's applied in rev 1680533

> Set port via property file when launching standalone
> ----------------------------------------------------
>
>                 Key: SLING-4672
>                 URL: https://issues.apache.org/jira/browse/SLING-4672
>             Project: Sling
>          Issue Type: Improvement
>          Components: Launchpad
>    Affects Versions: Launchpad Base 2.5.0
>            Reporter: Kevin A. Roll
>            Assignee: Carsten Ziegeler
>            Priority: Minor
>             Fix For: Launchpad Base 2.6.2
>
>
> Currently it is not possible to launch the Sling standalone jar and configure the port via the sling.properties/sling_install.properties file. This is because MainDelegate always puts a value in the property map for the port and then overrides Sling.loadConfigProperties(), passing this value. 
> Carsten suggested on the mailing list:
> "This is a little bit tricky, in general it should be possible to just rely on sling properties. The general mechanism is that you can override values of sling properties when launching, e.g. by specifying the -D option. Unfortunately, the default port is set by the MainDelegate in the same way. Therefore the underlying mechanism which processes sling properties thinks that this is an override and overwrites the sling properties setting. If I see it correctly, the port is the only property that has this special handling. I guess we could solve this by defining the default port in the sling.properties file from base and removing the special handling in the launchpad code."



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