Windows Service parameters contains a break line parameter

Summary

Windows Service parameters contains a break line parameter.
As a result, any parameter defined after the break line will not be registered by Crowd.

Environment

• Crowd 3.3.3
• Windows Service

Steps to Reproduce

1. Install Crowd 3.3.3
2. Follow the instructions in Installing Crowd as a Windows Service to install Crowd as a service
3. Check the registry

Expected Results

There is no break line

Actual Results

There is a break line in the parameters

Notes

1. Everything that's defined after the break line does not get registered in Crowd.
2. From the example above, currently the encoding setting "-Dfile.encoding=UTF-8" will not be registered in Crowd
3. The following was taken from application.xml taken from Crowd that was started by through the service:

   <virtual-machine-arguments>-Dcatalina.home=D:\Crowd\Install\atlassian-crowd-3.3.3\apache-tomcat -Dcatalina.base=D:\Crowd\Install\atlassian-crowd-3.3.3\apache-tomcat -Dignore.endorsed.dirs=D:\Crowd\Install\atlassian-crowd-3.3.3\apache-tomcat\endorsed -Djava.io.tmpdir=D:\Crowd\Install\atlassian-crowd-3.3.3\apache-tomcat\temp -Djava.util.logging.manager=org.apache.juli.ClassLoaderLogManager -Djava.util.logging.config.file=D:\Crowd\Install\atlassian-crowd-3.3.3\apache-tomcat\conf\logging.properties exit abort -Xms128m -Xmx256m</virtual-machine-arguments>
   

4. While the following was taken from application.xml taken from Crowd that was started manually :

   <virtual-machine-arguments>-Djava.util.logging.config.file=D:\Crowd\Install\atlassian-crowd-3.3.3\apache-tomcat\conf\logging.properties -Djava.util.logging.manager=org.apache.juli.ClassLoaderLogManager -Xms128m -Xmx512m -Dfile.encoding=UTF-8 -Djdk.tls.ephemeralDHKeySize=2048 -Djava.protocol.handler.pkgs=org.apache.catalina.webresources -Dignore.endorsed.dirs= -Dcatalina.base=D:\Crowd\Install\atlassian-crowd-3.3.3\apache-tomcat -Dcatalina.home=D:\Crowd\Install\atlassian-crowd-3.3.3\apache-tomcat -Djava.io.tmpdir=D:\Crowd\Install\atlassian-crowd-3.3.3\apache-tomcat\temp</virtual-machine-arguments>
   

Take note that the parameter -Dfile.encoding=UTF-8 is not included in the Crowd that's started by the service

Workaround

Deleting the break line from the registry manually helped