Menu
▾
▴
RE: [Springframework-developer] Summary of pending points
|
From: <jue...@we...> - 2003-07-25 14:36:13
|
Just committed! -----Original Message----- From: j=FCrgen h=F6ller [werk3AT]=20 Sent: Friday, July 25, 2003 4:01 PM To: jp....@ti... Cc: alef; springframework-developer Subject: RE: [Springframework-developer] Summary of pending points That's what I assumed. I've already added a special workaround for this by simply using an own = WebUtils.getRequestURI(request) helper method instead of = request.getRequestURI(). The rest of the WebUtils.getPathXXX = implementations can stay exactly the same that way. Juergen -----Original Message----- From: jp....@ti... [mailto:jp....@ti...] Sent: Friday, July 25, 2003 4:00 PM To: j=FCrgen h=F6ller [werk3AT] Cc: alef; springframework-developer Subject: RE: [Springframework-developer] Summary of pending points I confirm it's a JBoss/Jetty issue. Probably it's reported on JBoss = lists. Still the countries demo is affected.=20 Jean-Pierre ---------- Initial Header ----------- From : j=FCrgen h=F6ller [werk3AT] <jue...@we...> To : <al...@jt...>,<jp....@ti...> Cc : "springframework-developer" = <spr...@li...> Date : Fri, 25 Jul 2003 15:15:53 +0200 Subject : RE: [Springframework-developer] Summary of pending points 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. ********** 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. ------------------------------------------------------- This SF.Net email sponsored by: Free pre-built ASP.NET sites including Data Reports, E-commerce, Portals, and Forums are available now. Download today and enter to win an XBOX or Visual Studio .NET. http://aspnet.click-url.com/go/psa00100003ave/direct;at.aspnet_072303_01/= 01 _______________________________________________ Springframework-developer mailing list Spr...@li... https://lists.sourceforge.net/lists/listinfo/springframework-developer |
