You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by cs...@apache.org on 2017/03/21 14:26:42 UTC

cxf git commit: Add documentation, remove unused LogEventMapper interface

Repository: cxf
Updated Branches:
  refs/heads/master ae3ae2a25 -> 1a7411c27


Add documentation, remove unused LogEventMapper interface

Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/1a7411c2
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/1a7411c2
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/1a7411c2

Branch: refs/heads/master
Commit: 1a7411c270810507626a15d4a4cdc574d6ff62c3
Parents: ae3ae2a
Author: Christian Schneider <ch...@die-schneider.net>
Authored: Tue Mar 21 15:25:16 2017 +0100
Committer: Christian Schneider <ch...@die-schneider.net>
Committed: Tue Mar 21 15:26:30 2017 +0100

----------------------------------------------------------------------
 .../logging/event/DefaultLogEventMapper.java    |  2 +-
 .../cxf/ext/logging/event/LogEventMapper.java   | 25 --------------------
 .../ext/logging/event/LogMessageFormatter.java  |  2 +-
 .../ext/logging/event/PrettyLoggingFilter.java  |  4 ++++
 4 files changed, 6 insertions(+), 27 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf/blob/1a7411c2/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/DefaultLogEventMapper.java
----------------------------------------------------------------------
diff --git a/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/DefaultLogEventMapper.java b/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/DefaultLogEventMapper.java
index d76e5ac..27b9813 100644
--- a/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/DefaultLogEventMapper.java
+++ b/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/DefaultLogEventMapper.java
@@ -45,7 +45,7 @@ import org.apache.cxf.service.model.ServiceInfo;
 import org.apache.cxf.ws.addressing.AddressingProperties;
 import org.apache.cxf.ws.addressing.ContextUtils;
 
-public class DefaultLogEventMapper implements LogEventMapper {
+public class DefaultLogEventMapper {
     private static final Set<String> BINARY_CONTENT_MEDIA_TYPES;
     static {
         BINARY_CONTENT_MEDIA_TYPES = new HashSet<>();

http://git-wip-us.apache.org/repos/asf/cxf/blob/1a7411c2/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/LogEventMapper.java
----------------------------------------------------------------------
diff --git a/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/LogEventMapper.java b/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/LogEventMapper.java
deleted file mode 100644
index c187e3c..0000000
--- a/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/LogEventMapper.java
+++ /dev/null
@@ -1,25 +0,0 @@
-/**
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied. See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-package org.apache.cxf.ext.logging.event;
-
-import org.apache.cxf.message.Message;
-
-public interface LogEventMapper {
-    LogEvent map(Message message);
-}

http://git-wip-us.apache.org/repos/asf/cxf/blob/1a7411c2/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/LogMessageFormatter.java
----------------------------------------------------------------------
diff --git a/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/LogMessageFormatter.java b/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/LogMessageFormatter.java
index 0c628f7..fe872d2 100644
--- a/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/LogMessageFormatter.java
+++ b/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/LogMessageFormatter.java
@@ -24,7 +24,7 @@ import javax.xml.namespace.QName;
 import org.apache.cxf.common.util.StringUtils;
 
 /**
- *
+ * Formats a log message showing the most important meta data
  */
 public final class LogMessageFormatter {
     

http://git-wip-us.apache.org/repos/asf/cxf/blob/1a7411c2/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/PrettyLoggingFilter.java
----------------------------------------------------------------------
diff --git a/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/PrettyLoggingFilter.java b/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/PrettyLoggingFilter.java
index a3bc909..ccadc2f 100644
--- a/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/PrettyLoggingFilter.java
+++ b/rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/PrettyLoggingFilter.java
@@ -30,6 +30,10 @@ import org.apache.cxf.staxutils.StaxUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+/**
+ * Formats the log message with indents.
+ * If the log message was truncated then this filter might additionally cut incomplete tags
+ */
 public class PrettyLoggingFilter implements LogEventSender {
     private static final Logger LOG = LoggerFactory.getLogger(PrettyLoggingFilter.class);
     private LogEventSender next;