You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by ff...@apache.org on 2019/07/09 15:43:03 UTC

[cxf] branch 3.2.x-fixes updated: [CXF-8072]Loggers logs request twice in case of Fault

This is an automated email from the ASF dual-hosted git repository.

ffang pushed a commit to branch 3.2.x-fixes
in repository https://gitbox.apache.org/repos/asf/cxf.git


The following commit(s) were added to refs/heads/3.2.x-fixes by this push:
     new 3bff261  [CXF-8072]Loggers logs request twice in case of Fault
3bff261 is described below

commit 3bff2611dd318f11c4d58daa0a6de54549f50449
Author: Freeman Fang <fr...@gmail.com>
AuthorDate: Tue Jul 9 11:39:11 2019 -0400

    [CXF-8072]Loggers logs request twice in case of Fault
    
    (cherry picked from commit 5fe1123ec0c43c5345e76731f72dd802cae4c03d)
---
 .../org/apache/cxf/ext/logging/LoggingInInterceptor.java | 16 ----------------
 1 file changed, 16 deletions(-)

diff --git a/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/LoggingInInterceptor.java b/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/LoggingInInterceptor.java
index 8d06c7d..9ab3edf 100644
--- a/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/LoggingInInterceptor.java
+++ b/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/LoggingInInterceptor.java
@@ -35,7 +35,6 @@ import org.apache.cxf.interceptor.Fault;
 import org.apache.cxf.io.CachedOutputStream;
 import org.apache.cxf.io.CachedWriter;
 import org.apache.cxf.message.Message;
-import org.apache.cxf.phase.AbstractPhaseInterceptor;
 import org.apache.cxf.phase.Phase;
 import org.apache.cxf.phase.PhaseInterceptor;
 
@@ -44,20 +43,6 @@ import org.apache.cxf.phase.PhaseInterceptor;
  */
 @NoJSR250Annotations
 public class LoggingInInterceptor extends AbstractLoggingInterceptor {
-    class LoggingInFaultInterceptor extends AbstractPhaseInterceptor<Message> {
-        LoggingInFaultInterceptor() {
-            super(Phase.RECEIVE);
-        }
-
-        @Override
-        public void handleMessage(Message message) throws Fault {
-        }
-
-        @Override
-        public void handleFault(Message message) throws Fault {
-            LoggingInInterceptor.this.handleMessage(message);
-        }
-    }
 
     public LoggingInInterceptor() {
         this(new Slf4jVerboseEventSender());
@@ -74,7 +59,6 @@ public class LoggingInInterceptor extends AbstractLoggingInterceptor {
     public Collection<PhaseInterceptor<? extends Message>> getAdditionalInterceptors() {
         Collection<PhaseInterceptor<? extends Message>> ret = new ArrayList<>();
         ret.add(new WireTapIn(getWireTapLimit(), threshold));
-        ret.add(new LoggingInFaultInterceptor());
         return ret;
     }