You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by se...@apache.org on 2013/02/27 12:30:42 UTC

svn commit: r1450744 - in /cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport: http/AbstractHTTPDestination.java servlet/AbstractHTTPServlet.java

Author: sergeyb
Date: Wed Feb 27 11:30:41 2013
New Revision: 1450744

URL: http://svn.apache.org/r1450744
Log:
[CXF-4860] Keeping redirection status as HTTP attribute instead of parameter

Modified:
    cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java
    cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractHTTPServlet.java

Modified: cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java?rev=1450744&r1=1450743&r2=1450744&view=diff
==============================================================================
--- cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java (original)
+++ cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java Wed Feb 27 11:30:41 2013
@@ -233,7 +233,7 @@ public abstract class AbstractHTTPDestin
             LOG.fine("Get the message from the request for processing");
         }
 
-        copyKnownRequestParameters(req, inMessage);
+        copyKnownRequestAttributes(req, inMessage);
         
         try {    
             incomingObserver.onMessage(inMessage);
@@ -247,8 +247,8 @@ public abstract class AbstractHTTPDestin
  
     }
 
-    private void copyKnownRequestParameters(HttpServletRequest request, Message message) {
-        message.put(SERVICE_REDIRECTION, request.getParameter(SERVICE_REDIRECTION));
+    private void copyKnownRequestAttributes(HttpServletRequest request, Message message) {
+        message.put(SERVICE_REDIRECTION, request.getAttribute(SERVICE_REDIRECTION));
     }
     
     protected void setupMessage(Message inMessage,

Modified: cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractHTTPServlet.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractHTTPServlet.java?rev=1450744&r1=1450743&r2=1450744&view=diff
==============================================================================
--- cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractHTTPServlet.java (original)
+++ cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractHTTPServlet.java Wed Feb 27 11:30:41 2013
@@ -353,11 +353,11 @@ public abstract class AbstractHTTPServle
         }
         
         @Override
-        public String getParameter(String name) {
+        public Object getAttribute(String name) {
             if (AbstractHTTPDestination.SERVICE_REDIRECTION.equals(name)) {
                 return "true";
             }
-            return super.getParameter(name);
+            return super.getAttribute(name);
         }
     }