DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25403>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25403

[PATCH] EncodeURLTransformer unconditionally creates session





------- Additional Comments From [EMAIL PROTECTED]  2003-12-12 08:29 -------
hi bernhard,

I can't see which use cases this patch could break. and it is totally unrelated
to bug 16246.

this if from the code:

...
    /**
     * check if encoding of URLs is neccessary.
     * 
     * THis is true if session object exists, and session-id   
     * was provided from URL, or session is new.
     * The result is stored in some instance variables
     */
    protected void checkForEncoding(Request request) {
        this.session = request.getSession();
        this.isEncodeURLNeeded = false;
        
        if ( null != this.session ) {
...

this is the current CVS code and always creates a session (per
request.getSession()). the patch merely changes request.getSession() to
request.getSession(false), so that no new session is created.

Reply via email to