[prev in list] [next in list] [prev in thread] [next in thread] 

List:       apache-bugdb
Subject:    mod_rewrite/5073: mod_rewrite drop the QUERY_STRING using the [P] flag on per-dir context
From:       Martin Zeh <martin.zeh () sat1 ! de>
Date:       1999-09-28 16:19:05
[Download RAW message or body]


>Number:         5073
>Category:       mod_rewrite
>Synopsis:       mod_rewrite drop the QUERY_STRING using the [P] flag on per-dir context
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    apache
>State:          open
>Class:          sw-bug
>Submitter-Id:   apache
>Arrival-Date:   Tue Sep 28 09:20:00 PDT 1999
>Last-Modified:
>Originator:     martin.zeh@sat1.de
>Organization:
apache
>Release:        1.3.9
>Environment:
SunOS obelix 5.5.1 Generic_103640-27 sun4m sparc SUNW,SPARCstation-20
egcs-2.9.5
>Description:
The mod_rewrite in the apache 1.3.9 strip off the QUERY_STRING if
the force proxy flag is used in the per-directory context.
I think the same bug is reported earlier but the author forgot
the per-directory context. If [P] is used in global context all
is fine.
>How-To-Repeat:
This configuration doesn't work with 1.3.9.
With apache 1.3.6 it worked well!

<Location /hack>
   RewriteEngine On
   RewriteRule ^.*/htdocs/hack(.*)$ http://hugo.sat1.de:88/hack$1 [P,L]
</Location>
>Fix:
I comment the line 1377 in mod_rewrite.c but I didn't check the side
effects! After this change the QUERY_STRING is appended

1376: if (r->args != NULL) {
1377:    /* Martin Zeh changed 28.09.1999 && r->uri == r->unparsed_uri) { */
1378:    /* see proxy_http:proxy_http_canon() */
1379:    r->filename = ap_pstrcat(r->pool, r->filename,
1380:    "?", r->args, NULL);
1381: }
>Audit-Trail:
>Unformatted:
[In order for any reply to be added to the PR database, you need]
[to include <apbugs@Apache.Org> in the Cc line and make sure the]
[subject line starts with the report component and number, with ]
[or without any 'Re:' prefixes (such as "general/1098:" or      ]
["Re: general/1098:").  If the subject doesn't match this       ]
[pattern, your message will be misfiled and ignored.  The       ]
["apbugs" address is not added to the Cc line of messages from  ]
[the database automatically because of the potential for mail   ]
[loops.  If you do not include this Cc, your reply may be ig-   ]
[nored unless you are responding to an explicit request from a  ]
[developer.  Reply only with text; DO NOT SEND ATTACHMENTS!     ]

[prev in list] [next in list] [prev in thread] [next in thread]