Modified: qpid/java/trunk/broker-plugins/derby-store/src/main/java/org/apache/qpid/server/store/derby/DerbyConfigurationStore.java URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/derby-store/src/main/java/org/apache/qpid/server/store/derby/DerbyConfigurationStore.java?rev=1689268&r1=1689267&r2=1689268&view=diff ============================================================================== --- qpid/java/trunk/broker-plugins/derby-store/src/main/java/org/apache/qpid/server/store/derby/DerbyConfigurationStore.java (original) +++ qpid/java/trunk/broker-plugins/derby-store/src/main/java/org/apache/qpid/server/store/derby/DerbyConfigurationStore.java Sun Jul 5 17:46:21 2015 @@ -159,10 +159,7 @@ public class DerbyConfigurationStore ext { if (storePath != null) { - if (LOGGER.isDebugEnabled()) - { - LOGGER.debug("Deleting store " + storePath); - } + LOGGER.debug("Deleting store {}", storePath); File configFile = new File(storePath); if (!FileUtils.delete(configFile, true))
Modified: qpid/java/trunk/broker-plugins/derby-store/src/main/java/org/apache/qpid/server/store/derby/DerbyMessageStore.java URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/derby-store/src/main/java/org/apache/qpid/server/store/derby/DerbyMessageStore.java?rev=1689268&r1=1689267&r2=1689268&view=diff ============================================================================== --- qpid/java/trunk/broker-plugins/derby-store/src/main/java/org/apache/qpid/server/store/derby/DerbyMessageStore.java (original) +++ qpid/java/trunk/broker-plugins/derby-store/src/main/java/org/apache/qpid/server/store/derby/DerbyMessageStore.java Sun Jul 5 17:46:21 2015 @@ -84,10 +84,7 @@ public class DerbyMessageStore extends A if (storePath != null) { - if (LOGGER.isDebugEnabled()) - { - LOGGER.debug("Deleting store " + storePath); - } + LOGGER.debug("Deleting store {}", storePath); File configFile = new File(storePath); if (!FileUtils.delete(configFile, true)) Modified: qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/servlet/rest/RestServlet.java URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/servlet/rest/RestServlet.java?rev=1689268&r1=1689267&r2=1689268&view=diff ============================================================================== --- qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/servlet/rest/RestServlet.java (original) +++ qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/servlet/rest/RestServlet.java Sun Jul 5 17:46:21 2015 @@ -885,10 +885,7 @@ public class RestServlet extends Abstrac { if (e instanceof AccessControlException) { - if (LOGGER.isDebugEnabled()) - { - LOGGER.debug("AccessControlException, sending " + HttpServletResponse.SC_FORBIDDEN, e); - } + LOGGER.debug("AccessControlException, sending {}", HttpServletResponse.SC_FORBIDDEN, e); response.setStatus(HttpServletResponse.SC_FORBIDDEN); } else Modified: qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/servlet/rest/SaslServlet.java URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/servlet/rest/SaslServlet.java?rev=1689268&r1=1689267&r2=1689268&view=diff ============================================================================== --- qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/servlet/rest/SaslServlet.java (original) +++ qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/servlet/rest/SaslServlet.java Sun Jul 5 17:46:21 2015 @@ -128,10 +128,8 @@ public class SaslServlet extends Abstrac { if(id == null) { - if(LOGGER.isDebugEnabled()) - { - LOGGER.debug("Creating SaslServer for mechanism: " + mechanism); - } + LOGGER.debug("Creating SaslServer for mechanism: {}", mechanism); + SaslServer saslServer = subjectCreator.createSaslServer(mechanism, request.getServerName(), null/*TODO*/); evaluateSaslResponse(request, response, session, saslResponse, saslServer, subjectCreator); } Modified: qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/session/LoginLogoutReporter.java URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/session/LoginLogoutReporter.java?rev=1689268&r1=1689267&r2=1689268&view=diff ============================================================================== --- qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/session/LoginLogoutReporter.java (original) +++ qpid/java/trunk/broker-plugins/management-http/src/main/java/org/apache/qpid/server/management/plugin/session/LoginLogoutReporter.java Sun Jul 5 17:46:21 2015 @@ -69,10 +69,7 @@ public class LoginLogoutReporter impleme private void reportLogin() { - if (LOGGER.isDebugEnabled()) - { - LOGGER.debug("User logging in : " + _principal); - } + LOGGER.debug("User logging in : {}", _principal); Subject.doAs(_subject, new PrivilegedAction<Void>() { @@ -87,10 +84,7 @@ public class LoginLogoutReporter impleme private void reportLogout() { - if (LOGGER.isDebugEnabled()) - { - LOGGER.debug("User logging out : " + _principal); - } + LOGGER.debug("User logging out : {}", _principal); Subject.doAs(_subject, new PrivilegedAction<Void>() { Modified: qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/JMXManagedObjectRegistry.java URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/JMXManagedObjectRegistry.java?rev=1689268&r1=1689267&r2=1689268&view=diff ============================================================================== --- qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/JMXManagedObjectRegistry.java (original) +++ qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/JMXManagedObjectRegistry.java Sun Jul 5 17:46:21 2015 @@ -416,10 +416,8 @@ public class JMXManagedObjectRegistry im final RMIServerSocketFactory ssf; if(useCustomRmiRegistry) { - if (_log.isDebugEnabled()) - { - _log.debug("Using registry-protecting RMIServerSocketFactory"); - } + _log.debug("Using registry-protecting RMIServerSocketFactory"); + ssf = new RegistryProtectingRMIServerSocketFactory(); } else Modified: qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/JMXManagementPluginImpl.java URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/JMXManagementPluginImpl.java?rev=1689268&r1=1689267&r2=1689268&view=diff ============================================================================== --- qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/JMXManagementPluginImpl.java (original) +++ qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/JMXManagementPluginImpl.java Sun Jul 5 17:46:21 2015 @@ -255,18 +255,12 @@ public class JMXManagementPluginImpl { for (MBeanProvider provider : _mBeanProviders) { - if(LOGGER.isDebugEnabled()) - { - LOGGER.debug("Consulting mbean provider : " + provider + " for child : " + child); - } + LOGGER.debug("Consulting mbean provider : {} for child : {}", provider, child); ManagedObject mBean = null; if (provider.isChildManageableByMBean(child) && !providerMBeanExists(child, provider)) { - if(LOGGER.isDebugEnabled()) - { - LOGGER.debug("Provider of type " + provider.getType() + " will create mbean for " + child); - } + LOGGER.debug("Provider of type {} will create mbean for {}", provider.getType() , child); mBean = provider.createMBean(child, registry); if (mBean != null) Modified: qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/ManagementLogonLogoffReporter.java URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/ManagementLogonLogoffReporter.java?rev=1689268&r1=1689267&r2=1689268&view=diff ============================================================================== --- qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/ManagementLogonLogoffReporter.java (original) +++ qpid/java/trunk/broker-plugins/management-jmx/src/main/java/org/apache/qpid/server/jmx/ManagementLogonLogoffReporter.java Sun Jul 5 17:46:21 2015 @@ -60,10 +60,7 @@ public class ManagementLogonLogoffReport final String connectionId = ((JMXConnectionNotification) notification).getConnectionId(); final String type = notification.getType(); - if (LOGGER.isDebugEnabled()) - { - LOGGER.debug("Notification connectionId : " + connectionId + " type : " + type); - } + LOGGER.debug("Notification connectionId : {} type : {}", connectionId, type); Subject subject = _usernameAccessor.getSubjectConnectionId(connectionId); if(subject == null) --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
