svn commit: r1543424 - in /cxf/trunk: core/src/main/java/org/apache/cxf/interceptor/ rt/transports/local/src/main/java/org/apache/cxf/transport/local/

[sergeyb]

Author: sergeyb
Date: Tue Nov 19 13:27:05 2013
New Revision: 1543424

URL: http://svn.apache.org/r1543424
Log:
[CXF-5377] More updates for Logging interceptors to better report the request 
URI for non-HTTP transports

Modified:
    
cxf/trunk/core/src/main/java/org/apache/cxf/interceptor/LoggingInInterceptor.java
    
cxf/trunk/core/src/main/java/org/apache/cxf/interceptor/LoggingOutInterceptor.java
    
cxf/trunk/rt/transports/local/src/main/java/org/apache/cxf/transport/local/LocalTransportFactory.java

Modified: 
cxf/trunk/core/src/main/java/org/apache/cxf/interceptor/LoggingInInterceptor.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/core/src/main/java/org/apache/cxf/interceptor/LoggingInInterceptor.java?rev=1543424&r1=1543423&r2=1543424&view=diff
==============================================================================
--- 
cxf/trunk/core/src/main/java/org/apache/cxf/interceptor/LoggingInInterceptor.java
 (original)
+++ 
cxf/trunk/core/src/main/java/org/apache/cxf/interceptor/LoggingInInterceptor.java
 Tue Nov 19 13:27:05 2013
@@ -23,6 +23,7 @@ import java.io.PrintWriter;
 import java.io.Reader;
 import java.util.logging.Level;
 import java.util.logging.Logger;
+
 import org.apache.cxf.common.injection.NoJSR250Annotations;
 import org.apache.cxf.common.logging.LogUtils;
 import org.apache.cxf.helpers.IOUtils;
@@ -118,6 +119,15 @@ public class LoggingInInterceptor extend
         String uri = (String)message.get(Message.REQUEST_URL);
         if (uri == null) {
             uri = (String)message.get(Message.REQUEST_URI);
+            if (uri != null && uri.startsWith("/")) {
+                String address = (String)message.get(Message.ENDPOINT_ADDRESS);
+                if (address != null && !address.startsWith(uri)) {
+                    if (address.endsWith("/") && address.length() > 1) {
+                        address = address.substring(0, address.length()); 
+                    }
+                    uri = address + uri;
+                }
+            }
         }
         if (uri != null) {
             buffer.getAddress().append(uri);

Modified: 
cxf/trunk/core/src/main/java/org/apache/cxf/interceptor/LoggingOutInterceptor.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/core/src/main/java/org/apache/cxf/interceptor/LoggingOutInterceptor.java?rev=1543424&r1=1543423&r2=1543424&view=diff
==============================================================================
--- 
cxf/trunk/core/src/main/java/org/apache/cxf/interceptor/LoggingOutInterceptor.java
 (original)
+++ 
cxf/trunk/core/src/main/java/org/apache/cxf/interceptor/LoggingOutInterceptor.java
 Tue Nov 19 13:27:05 2013
@@ -115,6 +115,13 @@ public class LoggingOutInterceptor exten
         String address = (String)message.get(Message.ENDPOINT_ADDRESS);
         if (address != null) {
             buffer.getAddress().append(address);
+            String uri = (String)message.get(Message.REQUEST_URI);
+            if (uri != null && !address.startsWith(uri)) {
+                if (!address.endsWith("/") && !uri.startsWith("/")) {
+                    buffer.getAddress().append("/");
+                }
+                buffer.getAddress().append(uri);
+            }
         }
         String ct = (String)message.get(Message.CONTENT_TYPE);
         if (ct != null) {

Modified: 
cxf/trunk/rt/transports/local/src/main/java/org/apache/cxf/transport/local/LocalTransportFactory.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/transports/local/src/main/java/org/apache/cxf/transport/local/LocalTransportFactory.java?rev=1543424&r1=1543423&r2=1543424&view=diff
==============================================================================
--- 
cxf/trunk/rt/transports/local/src/main/java/org/apache/cxf/transport/local/LocalTransportFactory.java
 (original)
+++ 
cxf/trunk/rt/transports/local/src/main/java/org/apache/cxf/transport/local/LocalTransportFactory.java
 Tue Nov 19 13:27:05 2013
@@ -90,6 +90,7 @@ public class LocalTransportFactory exten
         messageIncludeProperties.add(Message.ACCEPT_CONTENT_TYPE);
         messageIncludeProperties.add(Message.RESPONSE_CODE);
         messageIncludeProperties.add(Message.REQUEST_URI);
+        messageIncludeProperties.add(Message.ENDPOINT_ADDRESS);
         messageIncludeProperties.add(Message.HTTP_REQUEST_METHOD);
     }