Hai, Great job!!! If you can add this patch to jira.opensymphony.com, we'll be sure to get to it as soon as we can.
-Pat ----- Original Message ----- From: "Hai Pham" <[EMAIL PROTECTED]> To: "WebWork-Dev" <[EMAIL PROTECTED]> Sent: Thursday, October 31, 2002 9:35 PM Subject: [OS-webwork] [Patch] ServletDispatcher.getActionName brokeh when '/' is present in query param. > Hi all, > > This bug is present from webwork 1.2.1. The dispatcher > will not be able to pull out the action name if char. > '/' is present in query param. > > An example: > <view > name="success">/Redirect.jspa?url=http://abc.com</view> > > The current code looks for the last '/' char and then > get the action name from that position, which is > obviously wrong in this case. > > This is the diff for > webwork.dispatcher.ServletDispatcher in release_1_2_1 > tag: > > E:\external_projects\webwork1.2.1\webwork\src\main\webwork\dispatcher>cvs > diff > cvs server: Diffing . > Index: ServletDispatcher.java > =================================================================== > RCS file: > /cvsroot/opensymphony/webwork/src/main/webwork/dispatcher/ServletDispa > tcher.java,v > retrieving revision 1.43 > diff -r1.43 ServletDispatcher.java > 421c421,424 > < int beginIdx = name.lastIndexOf("/"); > --- > > int questionIdx = name.indexOf('?'); > > int beginIdx = (questionIdx == -1 > > ? name.lastIndexOf("/") > > : name.lastIndexOf('/', questionIdx)); > > Thanks. > > ===== > --------------------------------------------- > Hai Pham Quang > --------------------------------------------- > > __________________________________________________ > Do you Yahoo!? > HotJobs - Search new jobs daily now > http://hotjobs.yahoo.com/ > > > ------------------------------------------------------- > This sf.net email is sponsored by: See the NEW Palm > Tungsten T handheld. Power & Color in a compact size! > http://ads.sourceforge.net/cgi-bin/redirect.pl?palm0001en > _______________________________________________ > Opensymphony-webwork mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/opensymphony-webwork ------------------------------------------------------- This sf.net email is sponsored by: See the NEW Palm Tungsten T handheld. Power & Color in a compact size! http://ads.sourceforge.net/cgi-bin/redirect.pl?palm0001en _______________________________________________ Opensymphony-webwork mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/opensymphony-webwork
