pvdung / urlrewritefilter

Automatically exported from code.google.com/p/urlrewritefilter
Other
0 stars 0 forks source link

ModRewriteConfLoader QSA support #117

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Enable mod_rewrite config style usage
2. Add rule 'RewriteRule /app http://localhost/app2 [P,QSA]'
3. There is a message in log 'qsappend flag [QSA] not supported'
4. Urls which are requested as /app/page?some=parameter
are get proxied as http://localhost/app2/page
i.e. without query parameters.

What is the expected output? What do you see instead?
- No such message,
- Query string is transfered to target page/site.

What version of the product are you using? On what operating system?
- 4.0.3

Please provide any additional information below.

The functionality is here, but not activated 
in ModRewriteConfLoader for some reason.

The patch is trivial, I am including it.
Patch made against trunk.

Original issue reported on code.google.com by rusla...@gmail.com on 7 Aug 2012 at 11:52

Attachments:

GoogleCodeExporter commented 9 years ago
When will 4.0.5 be released?

Original comment by marsman...@googlemail.com on 7 Aug 2012 at 6:09

GoogleCodeExporter commented 9 years ago
I also get this error. QSA is documented in the manual. Why am I getting this 
error?

Original comment by tra...@traviswebb.com on 23 Sep 2012 at 6:19

GoogleCodeExporter commented 9 years ago
Because it is working ok if you use urlrewritefilter
native config file. Not working only if rules expressed in mod_rewrite syntax.

Original comment by rusla...@gmail.com on 23 Sep 2012 at 8:49

GoogleCodeExporter commented 9 years ago
https://code.google.com/p/urlrewritefilter/issues/detail?id=31 Added support 
for this... looks like it only added it to the xml version and not the rewrite 
version?

Original comment by billapep...@gmail.com on 14 May 2014 at 5:49

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Here is a patch (including a test)

Original comment by billapep...@gmail.com on 14 May 2014 at 6:32

Attachments: