This is an automated email from the ASF dual-hosted git repository.
remm pushed a commit to branch 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.1.x by this push:
new 0ee5484741 Differentiate trace and debug log levels
0ee5484741 is described below
commit 0ee548474112ca48aabc5c9c67f8e67461b8c556
Author: remm <r...@apache.org>
AuthorDate: Fri Feb 2 13:51:40 2024 +0100
Differentiate trace and debug log levels
---
.../catalina/core/ApplicationDispatcher.java | 8 +--
.../org/apache/catalina/core/AsyncContextImpl.java | 28 +++++-----
.../apache/catalina/core/LocalStrings.properties | 3 +-
.../catalina/core/NamingContextListener.java | 4 +-
java/org/apache/catalina/core/StandardContext.java | 64 +++++++++++-----------
.../apache/catalina/core/StandardHostValve.java | 14 ++---
java/org/apache/catalina/core/StandardWrapper.java | 4 +-
7 files changed, 63 insertions(+), 62 deletions(-)
diff --git a/java/org/apache/catalina/core/ApplicationDispatcher.java
b/java/org/apache/catalina/core/ApplicationDispatcher.java
index 5672ebf823..149ba52ca1 100644
--- a/java/org/apache/catalina/core/ApplicationDispatcher.java
+++ b/java/org/apache/catalina/core/ApplicationDispatcher.java
@@ -347,8 +347,8 @@ final class ApplicationDispatcher implements
AsyncDispatcher, RequestDispatcher
}
// This is not a real close in order to support error processing
- if (wrapper.getLogger().isDebugEnabled()) {
- wrapper.getLogger().debug(" Disabling the response for further
output");
+ if (wrapper.getLogger().isTraceEnabled()) {
+ wrapper.getLogger().trace(" Disabling the response for further
output");
}
if (response instanceof ResponseFacade) {
@@ -356,9 +356,9 @@ final class ApplicationDispatcher implements
AsyncDispatcher, RequestDispatcher
} else {
// Servlet SRV.6.2.2. The Request/Response may have been wrapped
// and may no longer be instance of RequestFacade
- if (wrapper.getLogger().isDebugEnabled()) {
+ if (wrapper.getLogger().isTraceEnabled()) {
wrapper.getLogger()
- .debug(" The Response is vehiculed using a wrapper: "
+ response.getClass().getName());
+ .trace(" The Response is vehiculed using a wrapper: "
+ response.getClass().getName());
}
// Close anyway
diff --git a/java/org/apache/catalina/core/AsyncContextImpl.java
b/java/org/apache/catalina/core/AsyncContextImpl.java
index b713f71f10..0840b7a61e 100644
--- a/java/org/apache/catalina/core/AsyncContextImpl.java
+++ b/java/org/apache/catalina/core/AsyncContextImpl.java
@@ -78,7 +78,7 @@ public class AsyncContextImpl implements AsyncContext,
AsyncContextCallback {
private final AtomicBoolean hasOnErrorReturned = new AtomicBoolean(false);
public AsyncContextImpl(Request request) {
- if (log.isDebugEnabled()) {
+ if (log.isTraceEnabled()) {
logDebug("Constructor");
}
this.request = request;
@@ -86,7 +86,7 @@ public class AsyncContextImpl implements AsyncContext,
AsyncContextCallback {
@Override
public void complete() {
- if (log.isDebugEnabled()) {
+ if (log.isTraceEnabled()) {
logDebug("complete ");
}
check();
@@ -95,8 +95,8 @@ public class AsyncContextImpl implements AsyncContext,
AsyncContextCallback {
@Override
public void fireOnComplete() {
- if (log.isDebugEnabled()) {
- log.debug(sm.getString("asyncContextImpl.fireOnComplete"));
+ if (log.isTraceEnabled()) {
+ log.trace(sm.getString("asyncContextImpl.fireOnComplete"));
}
List<AsyncListenerWrapper> listenersCopy = new ArrayList<>(listeners);
@@ -125,8 +125,8 @@ public class AsyncContextImpl implements AsyncContext,
AsyncContextCallback {
Context context = this.context;
if (result.get()) {
- if (log.isDebugEnabled()) {
- log.debug(sm.getString("asyncContextImpl.fireOnTimeout"));
+ if (log.isTraceEnabled()) {
+ log.trace(sm.getString("asyncContextImpl.fireOnTimeout"));
}
ClassLoader oldCL = context.bind(false, null);
try {
@@ -179,7 +179,7 @@ public class AsyncContextImpl implements AsyncContext,
AsyncContextCallback {
@Override
public void dispatch(ServletContext servletContext, String path) {
synchronized (asyncContextLock) {
- if (log.isDebugEnabled()) {
+ if (log.isTraceEnabled()) {
logDebug("dispatch ");
}
check();
@@ -226,7 +226,7 @@ public class AsyncContextImpl implements AsyncContext,
AsyncContextCallback {
@Override
public void start(final Runnable run) {
- if (log.isDebugEnabled()) {
+ if (log.isTraceEnabled()) {
logDebug("start ");
}
check();
@@ -271,7 +271,7 @@ public class AsyncContextImpl implements AsyncContext,
AsyncContextCallback {
}
public void recycle() {
- if (log.isDebugEnabled()) {
+ if (log.isTraceEnabled()) {
logDebug("recycle ");
}
context = null;
@@ -312,8 +312,8 @@ public class AsyncContextImpl implements AsyncContext,
AsyncContextCallback {
List<AsyncListenerWrapper> listenersCopy = new
ArrayList<>(listeners);
listeners.clear();
- if (log.isDebugEnabled()) {
- log.debug(sm.getString("asyncContextImpl.fireOnStartAsync"));
+ if (log.isTraceEnabled()) {
+ log.trace(sm.getString("asyncContextImpl.fireOnStartAsync"));
}
for (AsyncListenerWrapper listener : listenersCopy) {
try {
@@ -333,7 +333,7 @@ public class AsyncContextImpl implements AsyncContext,
AsyncContextCallback {
}
protected void doInternalDispatch() throws ServletException, IOException {
- if (log.isDebugEnabled()) {
+ if (log.isTraceEnabled()) {
logDebug("intDispatch");
}
try {
@@ -392,8 +392,8 @@ public class AsyncContextImpl implements AsyncContext,
AsyncContextCallback {
request.getCoyoteRequest().action(ActionCode.ASYNC_ERROR, null);
if (fireOnError) {
- if (log.isDebugEnabled()) {
- log.debug(sm.getString("asyncContextImpl.fireOnError"));
+ if (log.isTraceEnabled()) {
+ log.trace(sm.getString("asyncContextImpl.fireOnError"));
}
AsyncEvent errorEvent =
new AsyncEvent(event.getAsyncContext(),
event.getSuppliedRequest(), event.getSuppliedResponse(), t);
diff --git a/java/org/apache/catalina/core/LocalStrings.properties
b/java/org/apache/catalina/core/LocalStrings.properties
index ec1126d4bf..7095750a91 100644
--- a/java/org/apache/catalina/core/LocalStrings.properties
+++ b/java/org/apache/catalina/core/LocalStrings.properties
@@ -260,7 +260,8 @@ standardHost.nullName=Host name is required
standardHost.problematicAppBase=Using an empty string for appBase on host
[{0}] will set it to CATALINA_BASE, which is a bad idea
standardHost.problematicLegacyAppBase=Using an empty string for legacyAppBase
on host [{0}] will set it to CATALINA_BASE, which is a bad idea
-standardHostValue.customStatusFailed=Custom error page [{0}] could not be
dispatched correctly
+standardHostValve.customStatusFailed=Custom error page [{0}] could not be
dispatched correctly
+standardHostValve.exception=Exception Processing [{0}]
standardPipeline.basic.start=Error starting new basic Valve
standardPipeline.basic.stop=Error stopping old basic Valve
diff --git a/java/org/apache/catalina/core/NamingContextListener.java
b/java/org/apache/catalina/core/NamingContextListener.java
index 7013eb7d65..a81ff0a04c 100644
--- a/java/org/apache/catalina/core/NamingContextListener.java
+++ b/java/org/apache/catalina/core/NamingContextListener.java
@@ -496,8 +496,8 @@ public class NamingContextListener implements
LifecycleListener, PropertyChangeL
int i;
- if (log.isDebugEnabled()) {
- log.debug("Creating JNDI naming context");
+ if (log.isTraceEnabled()) {
+ log.trace("Creating JNDI naming context");
}
if (namingResources == null) {
diff --git a/java/org/apache/catalina/core/StandardContext.java
b/java/org/apache/catalina/core/StandardContext.java
index df80ff191f..0aa846988b 100644
--- a/java/org/apache/catalina/core/StandardContext.java
+++ b/java/org/apache/catalina/core/StandardContext.java
@@ -2125,8 +2125,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
@Override
public void setPublicId(String publicId) {
- if (log.isDebugEnabled()) {
- log.debug("Setting deployment descriptor public ID to '" +
publicId + "'");
+ if (log.isTraceEnabled()) {
+ log.trace("Setting deployment descriptor public ID to '" +
publicId + "'");
}
String oldPublicId = this.publicId;
@@ -4271,8 +4271,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
*/
public boolean filterStart() {
- if (getLogger().isDebugEnabled()) {
- getLogger().debug("Starting filters");
+ if (getLogger().isTraceEnabled()) {
+ getLogger().trace("Starting filters");
}
// Instantiate and record a FilterConfig for each defined filter
boolean ok = true;
@@ -4280,8 +4280,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
filterConfigs.clear();
for (Entry<String,FilterDef> entry : filterDefs.entrySet()) {
String name = entry.getKey();
- if (getLogger().isDebugEnabled()) {
- getLogger().debug(" Starting filter '" + name + "'");
+ if (getLogger().isTraceEnabled()) {
+ getLogger().trace(" Starting filter '" + name + "'");
}
try {
ApplicationFilterConfig filterConfig = new
ApplicationFilterConfig(this, entry.getValue());
@@ -4306,15 +4306,15 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
*/
public boolean filterStop() {
- if (getLogger().isDebugEnabled()) {
- getLogger().debug("Stopping filters");
+ if (getLogger().isTraceEnabled()) {
+ getLogger().trace("Stopping filters");
}
// Release all Filter and FilterConfig instances
synchronized (filterDefs) {
for (Entry<String,ApplicationFilterConfig> entry :
filterConfigs.entrySet()) {
- if (getLogger().isDebugEnabled()) {
- getLogger().debug(" Stopping filter '" + entry.getKey() +
"'");
+ if (getLogger().isTraceEnabled()) {
+ getLogger().trace(" Stopping filter '" + entry.getKey() +
"'");
}
ApplicationFilterConfig filterConfig = entry.getValue();
filterConfig.release();
@@ -4348,8 +4348,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
*/
public boolean listenerStart() {
- if (log.isDebugEnabled()) {
- log.debug("Configuring application event listeners");
+ if (log.isTraceEnabled()) {
+ log.trace("Configuring application event listeners");
}
// Instantiate the required listeners
@@ -4357,8 +4357,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
Object results[] = new Object[listeners.length];
boolean ok = true;
for (int i = 0; i < results.length; i++) {
- if (getLogger().isDebugEnabled()) {
- getLogger().debug(" Configuring event listener class '" +
listeners[i] + "'");
+ if (getLogger().isTraceEnabled()) {
+ getLogger().trace(" Configuring event listener class '" +
listeners[i] + "'");
}
try {
String listener = listeners[i];
@@ -4406,8 +4406,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
// Send application start events
- if (getLogger().isDebugEnabled()) {
- getLogger().debug("Sending application start events");
+ if (getLogger().isTraceEnabled()) {
+ getLogger().trace("Sending application start events");
}
// Ensure context is not null
@@ -4457,8 +4457,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
*/
public boolean listenerStop() {
- if (log.isDebugEnabled()) {
- log.debug("Sending application stop events");
+ if (log.isTraceEnabled()) {
+ log.trace("Sending application stop events");
}
boolean ok = true;
@@ -4644,8 +4644,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
@Override
protected void startInternal() throws LifecycleException {
- if (log.isDebugEnabled()) {
- log.debug("Starting " + getBaseName());
+ if (log.isTraceEnabled()) {
+ log.trace("Starting " + getBaseName());
}
// Send j2ee.state.starting notification
@@ -4669,8 +4669,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
// Add missing components as necessary
if (getResources() == null) { // (1) Required by Loader
- if (log.isDebugEnabled()) {
- log.debug("Configuring default Resources");
+ if (log.isTraceEnabled()) {
+ log.trace("Configuring default Resources");
}
try {
@@ -4715,8 +4715,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
}
// Standard container startup
- if (log.isDebugEnabled()) {
- log.debug("Processing standard container startup");
+ if (log.isTraceEnabled()) {
+ log.trace("Processing standard container startup");
}
@@ -4922,8 +4922,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
// Set available status depending upon startup success
if (ok) {
- if (log.isDebugEnabled()) {
- log.debug("Starting completed");
+ if (log.isTraceEnabled()) {
+ log.trace("Starting completed");
}
} else {
log.error(sm.getString("standardContext.startFailed", getName()));
@@ -5112,8 +5112,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
setCharsetMapper(null);
// Normal container shutdown processing
- if (log.isDebugEnabled()) {
- log.debug("Processing standard container shutdown");
+ if (log.isTraceEnabled()) {
+ log.trace("Processing standard container shutdown");
}
// JNDI resources are unbound in CONFIGURE_STOP_EVENT so stop
@@ -5179,8 +5179,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
// reset the instance manager
setInstanceManager(null);
- if (log.isDebugEnabled()) {
- log.debug("Stopping complete");
+ if (log.isTraceEnabled()) {
+ log.trace("Stopping complete");
}
}
@@ -5297,8 +5297,8 @@ public class StandardContext extends ContainerBase
implements Context, Notificat
postConstructMethods.clear();
preDestroyMethods.clear();
- if (log.isDebugEnabled()) {
- log.debug("resetContext " + getObjectName());
+ if (log.isTraceEnabled()) {
+ log.trace("resetContext " + getObjectName());
}
}
diff --git a/java/org/apache/catalina/core/StandardHostValve.java
b/java/org/apache/catalina/core/StandardHostValve.java
index a186dc2a53..dc966435bc 100644
--- a/java/org/apache/catalina/core/StandardHostValve.java
+++ b/java/org/apache/catalina/core/StandardHostValve.java
@@ -116,7 +116,7 @@ final class StandardHostValve extends ValveBase {
}
} catch (Throwable t) {
ExceptionUtils.handleThrowable(t);
- container.getLogger().error("Exception Processing " +
request.getRequestURI(), t);
+
container.getLogger().error(sm.getString("standardHostValve.exception",
request.getRequestURI()), t);
// If a new error occurred while trying to report a previous
// error allow the original error to be reported.
if (!response.isErrorReportRequired()) {
@@ -226,7 +226,7 @@ final class StandardHostValve extends ValveBase {
} catch (ClientAbortException e) {
// Ignore
} catch (IOException e) {
- container.getLogger().warn("Exception Processing " +
errorPage, e);
+
container.getLogger().warn(sm.getString("standardHostValve.exception",
errorPage), e);
}
}
}
@@ -289,7 +289,7 @@ final class StandardHostValve extends ValveBase {
try {
response.finishResponse();
} catch (IOException e) {
- container.getLogger().warn("Exception Processing " +
errorPage, e);
+
container.getLogger().warn(sm.getString("standardHostValve.exception",
errorPage), e);
}
}
}
@@ -322,8 +322,8 @@ final class StandardHostValve extends ValveBase {
*/
private boolean custom(Request request, Response response, ErrorPage
errorPage) {
- if (container.getLogger().isDebugEnabled()) {
- container.getLogger().debug("Processing " + errorPage);
+ if (container.getLogger().isTraceEnabled()) {
+ container.getLogger().trace("Processing " + errorPage);
}
try {
@@ -333,7 +333,7 @@ final class StandardHostValve extends ValveBase {
if (rd == null) {
container.getLogger()
-
.error(sm.getString("standardHostValue.customStatusFailed",
errorPage.getLocation()));
+
.error(sm.getString("standardHostValve.customStatusFailed",
errorPage.getLocation()));
return false;
}
@@ -371,7 +371,7 @@ final class StandardHostValve extends ValveBase {
} catch (Throwable t) {
ExceptionUtils.handleThrowable(t);
// Report our failure to process this custom page
- container.getLogger().error("Exception Processing " + errorPage,
t);
+
container.getLogger().error(sm.getString("standardHostValve.exception",
errorPage), t);
return false;
}
}
diff --git a/java/org/apache/catalina/core/StandardWrapper.java
b/java/org/apache/catalina/core/StandardWrapper.java
index a21a58e4dd..0b74d061de 100644
--- a/java/org/apache/catalina/core/StandardWrapper.java
+++ b/java/org/apache/catalina/core/StandardWrapper.java
@@ -643,8 +643,8 @@ public class StandardWrapper extends ContainerBase
implements ServletConfig, Wra
synchronized (this) {
if (instance == null) {
try {
- if (log.isDebugEnabled()) {
- log.debug("Allocating instance");
+ if (log.isTraceEnabled()) {
+ log.trace("Allocating instance");
}
instance = loadServlet();
newInstance = true;
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
