tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Remy Maucherat <r...@apache.org>
Subject Re: svn commit: r486005 - in /tomcat/container/tc5.5.x: catalina/src/share/org/apache/catalina/connector/Request.java catalina/src/share/org/apache/catalina/session/PersistentManagerBase.java webapps/docs/changelog.xml
Date Tue, 12 Dec 2006 12:23:34 GMT
markt@apache.org wrote:
>              if ((session != null) && !session.isValid())
>                  session = null;
>              if (session != null) {
> +                if(!requestedSessionId.equals(session.getId())) {
> +                    Cookie cookie = new Cookie(Globals.SESSION_COOKIE_NAME,
> +                            session.getIdInternal());
> +                    configureSessionCookie(cookie);
> +                    response.addCookie(cookie);
> +                }

I don't know if that's a good idea. It looks a bit risky. I think it 
should include && (getContext() != null) && getContext().getCookies().

Rémy

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message