Re: svn commit: r160675 - httpd/httpd/trunk/modules/proxy/mod_proxy_balancer.c

[Jim Jagielski]

On Apr 9, 2005, at 6:12 AM, [EMAIL PROTECTED] wrote:
Author: mturk
Date: Sat Apr  9 03:12:01 2005
New Revision: 160675
URL: http://svn.apache.org/viewcvs?view=revrev=160675
Log:
Add canon_handler for load balancer so that we have
query string passed to remote.
Beat me to it :) I should really change my Email reading prefs to
newest 1st :)

Re: svn commit: r160675 - httpd/httpd/trunk/modules/proxy/mod_proxy_balancer.c

[Jim Jagielski]

On Apr 9, 2005, at 6:12 AM, [EMAIL PROTECTED] wrote:
+
+if (strncasecmp(url, balancer:, 9) == 0) {
+url += 9;
+scheme = balancer;
+}
+else {
+return DECLINED;
+}
+
Hold on a sec... I need to test this, but by the time this
is called, haven't we rewritten the URL (via rewrite_url())
to remove the 'balancer::' prefix?

Re: svn commit: r160675 - httpd/httpd/trunk/modules/proxy/mod_proxy_balancer.c

[Mladen Turk]

Jim Jagielski wrote:
 +if (strncasecmp(url, balancer:, 9) == 0) {

 Hold on a sec... I need to test this, but by the time this
 is called, haven't we rewritten the URL (via rewrite_url())
 to remove the 'balancer::' prefix?

No. The canon_handler is called during fixup hook while
rewrite_url is called from proxy_handler::pre_request hook,
thus later.
Regards,
Mladen.