[java-opensaml COMMIT] /trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/handler/AbstractMessageHand...

[noreply at shibboleth.net]

Author: tzeller
Date: Mon Aug  4 13:53:57 2014
New Revision: 3987

URL: http://svn.shibboleth.net/view/java-opensaml?rev=3987&view=rev
Log:
IDP-456 Cleanup logging of exceptions with parameterized messages.

Modified:
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/handler/AbstractMessageHandler.java

Modified: trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/handler/AbstractMessageHandler.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/handler/AbstractMessageHandler.java?rev=3987&r1=3986&r2=3987&view=diff
==============================================================================
--- trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/handler/AbstractMessageHandler.java (original)
+++ trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/handler/AbstractMessageHandler.java Mon Aug  4 13:53:57 2014
@@ -54,8 +54,8 @@
                     doPostInvoke(messageContext, e);
                 } catch (Throwable t) {
                     LoggerFactory.getLogger(AbstractMessageHandler.class).warn(
-                            getLogPrefix() + " Unchecked exception/error thrown by doPostInvoke, "
-                                    + "superseding a MessageHandlerException ", e);
+                            "{} Unchecked exception/error thrown by doPostInvoke, "
+                                    + "superseding a MessageHandlerException ", getLogPrefix(), e);
                     t.addSuppressed(e);
                     throw t;
                 }
@@ -65,8 +65,8 @@
                     doPostInvoke(messageContext);
                 } catch (Throwable t2) {
                     LoggerFactory.getLogger(AbstractMessageHandler.class).warn(
-                            getLogPrefix() + " Unchecked exception/error thrown by doPostInvoke, "
-                                    + "superseding an unchecked exception/error ", t);
+                            "{} Unchecked exception/error thrown by doPostInvoke, "
+                                    + "superseding an unchecked exception/error ", getLogPrefix(), t);
                     t2.addSuppressed(t);
                     throw t2;
                 }