cxf-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dk...@apache.org
Subject svn commit: r1502617 - in /cxf/branches/2.7.x-fixes: rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/ServletController.java systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSUriInfoTest.java
Date Fri, 12 Jul 2013 17:11:42 GMT
Author: dkulp
Date: Fri Jul 12 17:11:41 2013
New Revision: 1502617

URL: http://svn.apache.org/r1502617
Log:
Merged revisions 1492687 via  git cherry-pick from
https://svn.apache.org/repos/asf/cxf/trunk

........
  r1492687 | sergeyb | 2013-06-13 10:34:07 -0400 (Thu, 13 Jun 2013) | 2 lines

  [CXF-5067,CXF-5068] Always setting the current endpoint URI as HTTP attribute too

........

Modified:
    cxf/branches/2.7.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/ServletController.java
    cxf/branches/2.7.x-fixes/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSUriInfoTest.java

Modified: cxf/branches/2.7.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/ServletController.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.7.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/ServletController.java?rev=1502617&r1=1502616&r2=1502617&view=diff
==============================================================================
--- cxf/branches/2.7.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/ServletController.java
(original)
+++ cxf/branches/2.7.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/ServletController.java
Fri Jul 12 17:11:41 2013
@@ -93,11 +93,11 @@ public class ServletController {
             // Using HTTP_PREFIX check is safe for ServletController
             // URI.create(ad).isRelative() can be used - a bit more expensive though
             if (ad != null && !ad.startsWith(HTTP_PREFIX)) {
-                if (disableAddressUpdates) {
-                    request.setAttribute("org.apache.cxf.transport.endpoint.address", 
-                                         base + ad);
-                } else {
-                    BaseUrlHelper.setAddress(d, base + ad);
+                String endpointAddress = base + ad;
+                request.setAttribute("org.apache.cxf.transport.endpoint.address", 
+                                     endpointAddress);
+                if (!disableAddressUpdates) {
+                    BaseUrlHelper.setAddress(d, endpointAddress);
                 }
             }
         }

Modified: cxf/branches/2.7.x-fixes/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSUriInfoTest.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.7.x-fixes/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSUriInfoTest.java?rev=1502617&r1=1502616&r2=1502617&view=diff
==============================================================================
--- cxf/branches/2.7.x-fixes/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSUriInfoTest.java
(original)
+++ cxf/branches/2.7.x-fixes/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSUriInfoTest.java
Fri Jul 12 17:11:41 2013
@@ -61,6 +61,7 @@ public class JAXRSUriInfoTest extends Ab
      */
     @Test
     public void testBasePathAndPathAndPathParam() throws Exception {
+        checkUriInfo("http://127.0.0.1:" + PORT + "/app/v1", "\"\"", "/");
         checkUriInfo("http://localhost:" + PORT + "/app/v1", "\"\"", "/");
         checkUriInfo("http://localhost:" + PORT + "/app/v1/", "\"\"", "/");
         checkUriInfo("http://localhost:" + PORT + "/app/v1/test", "\"test\"", "test");



Mime
View raw message