Apache
/
activemq-artemis
mirror of https://github.com/apache/activemq-artemis.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

ARTEMIS-2342 Use a URL instead of a URI for the logging.configuration property 

The property the log manager itself expects is a URL which could result in an invalid URI missing the // prefix after file:.
This commit is contained in:
James Perkins 2019-05-16 14:07:25 -07:00 committed by Clebert Suconic
parent 2ae732cfc8
commit 4fffc02778
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ActiveMQServerImpl.java
View File

@ -23,7 +23,6 @@ import java.io.IOException;
import java.io.PrintWriter;
import java.io.StringWriter;
import java.lang.management.ManagementFactory;
import java.net.URI;
import java.net.URL;
import java.security.AccessController;
import java.security.PrivilegedAction;
@ -2772,7 +2771,7 @@ public class ActiveMQServerImpl implements ActiveMQServer {
if (System.getProperty("logging.configuration") != null) {
try {
reloadManager.addCallback(new URI(System.getProperty("logging.configuration")).toURL(), new LoggingConfigurationFileReloader());
reloadManager.addCallback(new URL(System.getProperty("logging.configuration")), new LoggingConfigurationFileReloader());
} catch (Exception e) {
// a syntax error with the logging system property shouldn't prevent the server from starting
ActiveMQServerLogger.LOGGER.problemAddingConfigReloadCallback(System.getProperty("logging.configuration"), e);