RE: [Springframework-developer] Summary of pending points

[<jue...@we...>]

I've just tested the issue with Tomcat 4.0, Tomcat 4.1, Resin 2.1, and =
Orion 2.0.2: All of these extract the URL path correctly both with and =
without a URL-encoded session id. So I wonder what web container you're =
using - does JBoss/Jetty have an issue here?

The root cause is that the container incorrectly resolves =
request.getRequestURI so that it includes the ";" part in your case. =
Simply checking for ";" and cutting the URI before it in WebUtils should =
help, instead of explicitly checking for ";jsessionid".

Juergen


-----Original Message-----
From: Alef Arendsen (JTeam) [mailto:al...@jt...]
Sent: Friday, July 25, 2003 2:56 PM
To: jp....@ti...
Cc: 'springframework-developer'
Subject: RE: [Springframework-developer] Summary of pending points


Arghh, stupid,

Anyway, the modified WebUtils is now included...

Alef

-----Oorspronkelijk bericht-----
Van: jp....@ti... [mailto:jp....@ti...]=20
Verzonden: Friday, July 25, 2003 2:49 PM
Aan: alef
CC: springframework-developer
Onderwerp: RE: [Springframework-developer] Summary of pending points


Hi Alef,

It seems you joined twice the testSuite and not the Webutils itself.

I didn't, for now, take the time for searching a solution for the
";jsessionid" issue, but I am still tired with it. So, it's very good to
hear anyone works at this.

Regards,
Jean-Pierre=20

---------- Initial Header -----------

From      : spr...@li...
To          : "'springframework-developer'"
<spr...@li...>
Cc          :=20
Date      : Fri, 25 Jul 2003 13:28:59 +0200
Subject : RE: [Springframework-developer] Summary of pending points

There's an (for my situation major) issue with the WebUtils :(

I've got an UrlMapping saying **/*.view --> SomeController

http://localhost/site/external/login-login.view
works, while
http://localhost/site/external/login-login.view;jsessionid=3D2c4s7a1vds3g=
b
.jboss1
does not. But actually it should!

I've prepared a fix for the issue, which involves adding a
conditionality to the getPathWithinServletMapping() and the
getPathWithinApplication() methods (in WebUtils). I've been able to test
the getPathWithinApplication()-method (using the WebUtilsTestSuite) and
basically it works. However, the getPathWithinServletMapping() method I
can't really test, because in the MockServlet*** classes, there's no
MockServletMapping yet (and I haven't got the time to add this before
the 0.9.1 release).

The fix works however for my application (for both URL-rewriting
requests and non-URL-rewriting requests). All other tests still succeed.

But, since I'm not able to test everything enough and don't know the
rest of the web stuff enough to know what the impact is, I am kind of
reluctant to add it, with the 0.9.1 release coming next week...

Is there somebody that can find the time to figure out everything still
works (or just says, yeah, it's ok ;-)?? The modified WebUtils and the
WebUtilsTestSuite are attached to the email...

Cheers,

Alef


********** L'ADSL A 20 EUR/MOIS**********
Tiscali propose l'ADSL le moins cher du march=E9 : 20 EUR/mois et le =
modem
ADSL offert !=20
Pour profiter de cette offre exceptionnelle, cliquez ici :
http://register.tiscali.fr/adsl/ Offre soumise =E0 conditions.