DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=41698>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=41698





------- Additional Comments From [EMAIL PROTECTED]  2007-02-27 13:08 -------
(In reply to comment #5)
> Ruediger,
> 
> (In reply to comment #4)
> > Your patch does not work with mod_rewrite (RewriteRules with the P flag). 
> > Thus
> > it cannot be applied to the codebase. Currently I see no way to fix this, so
> > Nicks proposal is the only possibility currently. But maybe I am wrong and 
> > you
> > can come up with something that also works with mod_rewrite. Thus leaving 
> > this
> > report open as an Enhancement.
> 
> Could you please give the Apache configuration exempt for which my patch will
> break mod_rewrite: I will try to see if it can be fixed.

It does not break mod_rewrite, but it does not work with mod_rewrite, as after
applying your patch things will behave differently when you do proxying via
ProxyPass or mod_rewrite

To give you a simple example:

RewriteRule ^/(.*) http://backend.org/$1 [P]

does the same as

ProxyPass / http://backend.org/

But using the RewriteRule instead of ProxyPass will leave the Destination header
unchanged with your patch. The basic shortcoming of your patch is that it does
not apply all the transformations done to the original request URL to the
Destination header URL, so they get out of sync. You only apply the
transformations caused by ProxyPass. Most of these transformations are done
during the translate_name hook, but some of them are also done during the fixup
phase (mod_rewrite in directory context).

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to