You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by pe...@apache.org on 2010/11/13 12:00:02 UTC
svn commit: r1034729 - in
/wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http:
BufferedWebResponse.java IBufferedWebResponse.java
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);
}