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:39:46 UTC

[cxf] branch master 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 master
in repository https://gitbox.apache.org/repos/asf/cxf.git


The following commit(s) were added to refs/heads/master by this push:
     new 5fe1123  [CXF-8072]Loggers logs request twice in case of Fault
     new aa1fdd0  Merge branch 'master' of github.com:apache/cxf
5fe1123 is described below

commit 5fe1123ec0c43c5345e76731f72dd802cae4c03d
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
---
 .../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 fe69ec1..355e300 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
@@ -34,7 +34,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;
 
@@ -43,20 +42,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());
@@ -73,7 +58,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;
     }