Brian Akins wrote:
> Here is a trivial patch that will allow proxy_handler to run the
> request_status hook if pre_request fails. This is necessary if all
> balncer members are in an error state, so that other modules get a
If all workers are in error state, worker will be NULL. Are you sure that
the post_request hook that is run in the balancer case is prepared for such
a situation?
So it might be saver to do something like this to solve your problem.
Any comments from the proxy gurus?
Index: mod_proxy.c
===================================================================
--- mod_proxy.c (Revision 280422)
+++ mod_proxy.c (Arbeitskopie)
@@ -680,7 +680,7 @@
/* Try to obtain the most suitable worker */
access_status = ap_proxy_pre_request(&worker, &balancer, r, conf,
&url);
if (access_status != OK)
- return access_status;
+ goto run_request_status;
if (balancer && balancer->max_attempts_set && !max_attempts)
max_attempts = balancer->max_attempts;
/* firstly, try a proxy, unless a NoProxy directive is active */
@@ -774,6 +774,7 @@
}
}
+run_request_status:
proxy_run_request_status(&access_status, r);
return access_status;
[..cut..]
Regards
Rüdiger
