You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2014/04/26 21:33:54 UTC
svn commit: r1590285 - in /tomcat/tc7.0.x/trunk: ./
java/org/apache/tomcat/util/IntrospectionUtils.java
webapps/docs/changelog.xml
Author: markt
Date: Sat Apr 26 19:33:54 2014
New Revision: 1590285
URL: http://svn.apache.org/r1590285
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=56441
Raise the visibility of exceptions thrown when a problem is encountered calling a getter or setter on a component attribute from debug to warning.
Modified:
tomcat/tc7.0.x/trunk/ (props changed)
tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/IntrospectionUtils.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
Propchange: tomcat/tc7.0.x/trunk/
------------------------------------------------------------------------------
Merged /tomcat/trunk:r1590283
Modified: tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/IntrospectionUtils.java
URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/IntrospectionUtils.java?rev=1590285&r1=1590284&r2=1590285&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/IntrospectionUtils.java (original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/IntrospectionUtils.java Sat Apr 26 19:33:54 2014
@@ -401,18 +401,15 @@ public final class IntrospectionUtils {
} catch (IllegalArgumentException ex2) {
log.warn("IAE " + o + " " + name + " " + value, ex2);
} catch (SecurityException ex1) {
- if (log.isDebugEnabled())
- log.debug("IntrospectionUtils: SecurityException for " +
- o.getClass() + " " + name + "=" + value + ")", ex1);
+ log.warn("IntrospectionUtils: SecurityException for " +
+ o.getClass() + " " + name + "=" + value + ")", ex1);
} catch (IllegalAccessException iae) {
- if (log.isDebugEnabled())
- log.debug("IntrospectionUtils: IllegalAccessException for " +
- o.getClass() + " " + name + "=" + value + ")", iae);
+ log.warn("IntrospectionUtils: IllegalAccessException for " +
+ o.getClass() + " " + name + "=" + value + ")", iae);
} catch (InvocationTargetException ie) {
ExceptionUtils.handleThrowable(ie.getCause());
- if (log.isDebugEnabled())
- log.debug("IntrospectionUtils: InvocationTargetException for " +
- o.getClass() + " " + name + "=" + value + ")", ie);
+ log.warn("IntrospectionUtils: InvocationTargetException for " +
+ o.getClass() + " " + name + "=" + value + ")", ie);
}
return false;
}
@@ -450,18 +447,15 @@ public final class IntrospectionUtils {
} catch (IllegalArgumentException ex2) {
log.warn("IAE " + o + " " + name, ex2);
} catch (SecurityException ex1) {
- if (log.isDebugEnabled())
- log.debug("IntrospectionUtils: SecurityException for " +
- o.getClass() + " " + name + ")", ex1);
+ log.warn("IntrospectionUtils: SecurityException for " +
+ o.getClass() + " " + name + ")", ex1);
} catch (IllegalAccessException iae) {
- if (log.isDebugEnabled())
- log.debug("IntrospectionUtils: IllegalAccessException for " +
- o.getClass() + " " + name + ")", iae);
+ log.warn("IntrospectionUtils: IllegalAccessException for " +
+ o.getClass() + " " + name + ")", iae);
} catch (InvocationTargetException ie) {
ExceptionUtils.handleThrowable(ie.getCause());
- if (log.isDebugEnabled())
- log.debug("IntrospectionUtils: InvocationTargetException for " +
- o.getClass() + " " + name + ")");
+ log.warn("IntrospectionUtils: InvocationTargetException for " +
+ o.getClass() + " " + name + ")");
}
return null;
}
Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1590285&r1=1590284&r2=1590285&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Sat Apr 26 19:33:54 2014
@@ -125,6 +125,11 @@
Ensure that a TLD parser obtained from the cache has the correct value
of <code>blockExternal</code>. (markt)
</fix>
+ <fix>
+ <bug>56441</bug>: Raise the visibility of exceptions thrown when a
+ problem is encountered calling a getter or setter on a component
+ attribute from debug to warning. (markt)
+ </fix>
</changelog>
</subsection>
<subsection name="Coyote">
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org