geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From djen...@apache.org
Subject svn commit: r449349 - in /geronimo/server/trunk/modules/geronimo-jetty/src/main/java/org/apache/geronimo/jetty: GeronimoServletHttpRequest.java JettyWebAppContext.java
Date Sun, 24 Sep 2006 04:07:17 GMT
Author: djencks
Date: Sat Sep 23 21:07:16 2006
New Revision: 449349

URL: http://svn.apache.org/viewvc?view=rev&rev=449349
Log:
GERONIMO-2422 likely fix for losing session id

Modified:
    geronimo/server/trunk/modules/geronimo-jetty/src/main/java/org/apache/geronimo/jetty/GeronimoServletHttpRequest.java
    geronimo/server/trunk/modules/geronimo-jetty/src/main/java/org/apache/geronimo/jetty/JettyWebAppContext.java

Modified: geronimo/server/trunk/modules/geronimo-jetty/src/main/java/org/apache/geronimo/jetty/GeronimoServletHttpRequest.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-jetty/src/main/java/org/apache/geronimo/jetty/GeronimoServletHttpRequest.java?view=diff&rev=449349&r1=449348&r2=449349
==============================================================================
--- geronimo/server/trunk/modules/geronimo-jetty/src/main/java/org/apache/geronimo/jetty/GeronimoServletHttpRequest.java
(original)
+++ geronimo/server/trunk/modules/geronimo-jetty/src/main/java/org/apache/geronimo/jetty/GeronimoServletHttpRequest.java
Sat Sep 23 21:07:16 2006
@@ -42,6 +42,7 @@
     }
     
     public void setRequestedSessionId(String pathParams) {
+        requestedSessionId = null;
         if (servletHandler.isUsingCookies()) {
             Cookie[] cookies= request.getCookies();
             if (cookies!=null && cookies.length>0) {

Modified: geronimo/server/trunk/modules/geronimo-jetty/src/main/java/org/apache/geronimo/jetty/JettyWebAppContext.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-jetty/src/main/java/org/apache/geronimo/jetty/JettyWebAppContext.java?view=diff&rev=449349&r1=449348&r2=449349
==============================================================================
--- geronimo/server/trunk/modules/geronimo-jetty/src/main/java/org/apache/geronimo/jetty/JettyWebAppContext.java
(original)
+++ geronimo/server/trunk/modules/geronimo-jetty/src/main/java/org/apache/geronimo/jetty/JettyWebAppContext.java
Sat Sep 23 21:07:16 2006
@@ -320,12 +320,11 @@
         ServletHttpRequest request = (ServletHttpRequest) httpRequest.getWrapper();
         if (null == request) {
             request = new GeronimoServletHttpRequest(handler, null, httpRequest);
-            ((GeronimoServletHttpRequest) request).setRequestedSessionId(pathParams);
             ServletHttpResponse response = new ServletHttpResponse(request, httpResponse);
             httpRequest.setWrapper(request);
             httpResponse.setWrapper(response);
         }
-
+        ((GeronimoServletHttpRequest) request).setRequestedSessionId(pathParams);
         if (null != handleInterceptor) {
             handleInterceptor.handle(pathInContext, pathParams, httpRequest, httpResponse,
new EndHandleInterceptor());   
         } else {



Mime
View raw message