tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Thomas <ma...@apache.org>
Subject Re: svn commit: r944511 - /tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java
Date Sat, 15 May 2010 20:17:54 GMT
On 15/05/2010 00:14, rjung@apache.org wrote:
> Author: rjung
> Date: Fri May 14 23:14:51 2010
> New Revision: 944511
>
> URL: http://svn.apache.org/viewvc?rev=944511&view=rev
> Log:
> Check whether context is null.

-1 on this change. I'll be reverting it shortly.

1. It isn't necessary. There is a null check in 
ApplicationSessionCookieConfig.getSessionCookieName()

2. It bypasses the session cookie config

Mark



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


Mime
View raw message