Re: [Apc-aa-coders] [Apc-aa-general] rewriting actionapps urls

[Michael M. <mi...@gn...>]

Hi

I think I have found a fix for this but I wonder whether it breaks anything else. 

In some (probably rare) cases AA uses QUERY_STRING_UNESCAPED. In there the brackets of als get escape -- als\[bla\] and teh following parse_str call will ignore the als. 

I have made a change in the function shtml_query_string - util.php3, 162

    $ret_string = ($REQUEST_URI AND strpos($REQUEST_URI, '?')) ?
                            substr($REQUEST_URI, strpos($REQUEST_URI, '?')+1) :
                  ( isset($REDIRECT_QUERY_STRING_UNESCAPED)    ?
                            $REDIRECT_QUERY_STRING_UNESCAPED :
                            stripslashes($QUERY_STRING_UNESCAPED) );

And now it works. Can someone check this / check it into SVN if it's ok. I'm not sure I have SVN access.

thanks

mimo

On Thursday 08 February 2007 18:24, Michael Moritz wrote:
> Hi
>
> By now I'm thinking this is a bug in AA. We run  aa-2.10.0
>
> Using my RewriteRule and redirecting to phpinfo displays the correct als.
> Redirecting to view.php3 adds the two backslashes. I think what happens is
> that in case of being called by <!--#include="/apc-aa/view.php3"-->
> ActionApps tries to parse the $_SERVER['QUERY_STRING'] and gets it wrong
> for some reason.
>
> Michael
>
> On Wednesday 07 February 2007 11:34, Michael Moritz wrote:
> > Hi
> >
> > I am experimenting with mod_rewrite to make AA URLs more readable and
> > also parseable for log file analysers. Now I have run into what seems a
> > silly problem:
> >
> > If you have a rule like this
> >
> > RewriteRule ^/sub-440625$ /sub.shtml?als[cid]=440624&debug=1&nocache=1
> > [NE]
> >
> > it reaches AA/PHP as
> >
> > Starting view
> > ParseViewParameters: vid=XXX, query_string=cmd:set:als:Array ( [cid\] =>
> > 440624\
> >
> > Note the two trailing backslashes. It seems to be caused by the square
> > brackets
> >
> > Any ideas how to fix this?
> >
> > Thanks,
> >
> > Michael
> >
> > -------------------------------------------------------------------------
> > Using Tomcat but need to do more? Need to support web services, security?
> > Get stuff done quickly with pre-integrated technology to make your job
> > easier. Download IBM WebSphere Application Server v.1.0.1 based on Apache
> > Geronimo
> > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > _______________________________________________
> > "Did you get answers to your ActionApps-related queries? If yes, please
> > help the ActionApps community by uploading the answers onto appropriate
> > space in the ActionApps documentation wiki. See the *How to contribute*
> > section today http://actionapps.org/en/How_To_Contribute"
> > _______________________________________________
> > apc-aa-general mailing list
> > apc...@li...
> > https://lists.sourceforge.net/lists/listinfo/apc-aa-general
>
> -------------------------------------------------------------------------
> Using Tomcat but need to do more? Need to support web services, security?
> Get stuff done quickly with pre-integrated technology to make your job
> easier. Download IBM WebSphere Application Server v.1.0.1 based on Apache
> Geronimo
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> _______________________________________________
> Apc-aa-coders mailing list
> Apc...@li...
> https://lists.sourceforge.net/lists/listinfo/apc-aa-coders