Author: pete
Date: Sat Nov 13 11:00:01 2010
New Revision: 1034729

URL: http://svn.apache.org/viewvc?rev=1034729&view=rev
Log:
WICKET-3161: cookies that are set during a buffered web response will not be 
transferred over redirecting. 

The solution to this issue is not what I consider pretty so please take time to 
review and feel free to improve it.

Sorry for the last commit message (I intended to create a path but was clicking 
too fast)

The previous commit is part of WICKET-3161

Modified:
    
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/BufferedWebResponse.java
    
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/IBufferedWebResponse.java

Modified: 
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/BufferedWebResponse.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/BufferedWebResponse.java?rev=1034729&r1=1034728&r2=1034729&view=diff
==============================================================================
--- 
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/BufferedWebResponse.java
 (original)
+++ 
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/BufferedWebResponse.java
 Sat Nov 13 11:00:01 2010
@@ -47,7 +47,8 @@ public class BufferedWebResponse extends
         */
        public BufferedWebResponse(WebResponse originalResponse)
        {
-               // if original response had some cookies set we should not 
forget to transfer them
+               // if original response had some cookies set we should not 
forget 
+               // to transfer them to the current response
                if(originalResponse instanceof IBufferedWebResponse)
                        ((IBufferedWebResponse) 
originalResponse).transferCookies(this);
 

Modified: 
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/IBufferedWebResponse.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/IBufferedWebResponse.java?rev=1034729&r1=1034728&r2=1034729&view=diff
==============================================================================
--- 
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/IBufferedWebResponse.java
 (original)
+++ 
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/IBufferedWebResponse.java
 Sat Nov 13 11:00:01 2010
@@ -20,5 +20,10 @@ import org.apache.wicket.request.http.We
 
 interface IBufferedWebResponse
 {
+       /**
+        * transfer cookie operations (add + clear cookie) to given web response
+        *
+        * @param webResponse web response that should receive the current 
cookie operation
+        */
        void transferCookies(WebResponse webResponse);
 }


Reply via email to